Skip to content

Commit

Permalink
Merge pull request #43 from drakest123/bug_fixes
Browse files Browse the repository at this point in the history
Bug fixes
  • Loading branch information
SnowHydrology authored Jul 25, 2024
2 parents c4d0dd7 + cd876a2 commit 2752d88
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 5 deletions.
6 changes: 3 additions & 3 deletions INSTALL.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
1. Clone the online GitHub repository either from the main git repo source (https://github.com/NOAA-OWP/snow17.git) or your fork of it -- e.g., `git clone https://github.com/NOAA-OWP/snow17.git`
2. Change directory to the build directory: `cd snow17/build/`
3. Copy `Makefile` to `Makefile.local` (`cp Makefile Makefile.local`) and edit the local version to update the compiler and paths if necessary to match the resources on your system.
4. Run `make -f MakefileLocal` to compile the program and generate the executable, which will be found in `snow17/bin/`
4. Run `make -f Makefile.local` to compile the program and generate the executable, which will be found in `snow17/bin/`
5. There is a test case in the `snow17/test_cases/` directory. To run it, first cd to the directory (`cd ../test_cases/`) and unpack the example: `tar -xzvf ex1.tgz`.
6. `cd ex1/run/` to get to the run directory and then execute the shell script (`./runSnow17.csh`) or type `../../../bin/snow17.exe namelist.bmi.HHWM8` into your command line
7. Check the example in the output folder (`cd ../output/`)
Expand All @@ -12,8 +12,8 @@ TL;DR:
```
git clone https://github.com/NOAA-OWP/snow17.git
cd snow17/build/
cp Makefile MakefileLocal # edit to match your compiler and system resources
make -f MakefileLocal
cp Makefile Makefile.local # edit to match your compiler and system resources
make -f Makefile.local
cd ../test_cases/
tar -xzvf ex1.tgz
cd ex1/run/
Expand Down
2 changes: 1 addition & 1 deletion src/share/forcingType.f90
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ subroutine initForcing(this, namelist)
use defNamelist
implicit none

class(forcing_type), intent(out) :: this
class(forcing_type), intent(inout) :: this
type(namelist_type), intent(in) :: namelist

! -- variable allocations (time dim not needed since forcings are one-rec scalars)
Expand Down
1 change: 1 addition & 0 deletions src/share/ioModule.f90
Original file line number Diff line number Diff line change
Expand Up @@ -172,6 +172,7 @@ SUBROUTINE init_forcing_files(namelist, runinfo, parameters)

! --- code ------------------------------------------------------------------
print*, 'Initializing forcing files'
ios = 0
found_start = 0
do nh=1, runinfo%n_hrus

Expand Down
2 changes: 1 addition & 1 deletion src/share/modelVarType.f90
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ subroutine initModelVar(this, namelist)
implicit none

! define variables
class(modelvar_type), intent(out) :: this
class(modelvar_type), intent(inout) :: this
type(namelist_type), intent(in) :: namelist

! -- variable allocations (time dim not needed since forcings are one-rec scalars)
Expand Down
1 change: 1 addition & 0 deletions src/snow19/updt19.f
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ SUBROUTINE UPDT19(OWE,OSC,TWE,COVER,IUPWE,IUPSC,

CVK MODIFIED 4/00 BY V. KOREN: TWO NEW STATES ADDED
C.......................................
REAL LIQW
DIMENSION ADC(11)
C
C COMMON BLOCKS
Expand Down

0 comments on commit 2752d88

Please sign in to comment.