-
Notifications
You must be signed in to change notification settings - Fork 312
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add meta-types for landunit types #5
Labels
code health
improving internal code structure to make easier to maintain (sustainability)
Comments
billsacks
added
enhancement
new capability or improved behavior of existing capability
priority: low
Background task that doesn't need to be done right away.
labels
Dec 16, 2017
This was referenced Dec 16, 2017
billsacks
added
code health
improving internal code structure to make easier to maintain (sustainability)
and removed
enhancement
new capability or improved behavior of existing capability
labels
Feb 8, 2018
billsacks
removed
the
priority: low
Background task that doesn't need to be done right away.
label
Jan 16, 2019
Merged
mariuslam
pushed a commit
to NordicESMhub/ctsm
that referenced
this issue
Aug 26, 2019
removed unused patch logging variables, initialized site logging variables
mariuslam
pushed a commit
to NordicESMhub/ctsm
that referenced
this issue
Aug 26, 2019
Update to 10/10/2017 release
mariuslam
pushed a commit
to NordicESMhub/ctsm
that referenced
this issue
Aug 26, 2019
Bring sci.1.4.0_api.3.0. to NCAR fates-release
mariuslam
pushed a commit
to NordicESMhub/ctsm
that referenced
this issue
Aug 26, 2019
…to_merge coastal-veg parameter check fix
mariuslam
pushed a commit
to NordicESMhub/ctsm
that referenced
this issue
Aug 26, 2019
shorter lines
mariuslam
pushed a commit
to NordicESMhub/ctsm
that referenced
this issue
Aug 26, 2019
Fixed indexing on the maximum decomposition rate
mariuslam
pushed a commit
to NordicESMhub/ctsm
that referenced
this issue
Aug 26, 2019
…charfix removing call the char() for log output
billsacks
pushed a commit
to billsacks/ctsm
that referenced
this issue
Nov 7, 2019
Changes we discussed this morning, plus start adding unit tests
slevis-lmwg
referenced
this issue
in slevis-lmwg/ctsm
Aug 1, 2023
Fix testlist so options are under <test> rather than <machines>.
samsrabin
referenced
this issue
in samsrabin/CTSM
Apr 19, 2024
Fortran subroutines used for CFSR data source are added. The updated code can be compiled successfully on Hera.
AGonzalezNicolas
pushed a commit
to HPSCTerrSys/clm5_0
that referenced
this issue
Jun 27, 2024
Commit summary: * Trimmed model params file to show only the most important options * Removed user_nl arg from build_* methods * Created 'build_namelist' wrapper function * Changed config file to .toml format * Added optional param 'fill_missing_groups' to nml2str * Improved logging. Also separated build_seq_maps_rc() from gen_drv_in.py * Made build_* methods return a success flag (bool) and a status message * Auto-split lines in namelist that exceed 80 columns into multiple lines * Bump clm5nl-gen to v0.3. Also added a minimal README file.
AGonzalezNicolas
added a commit
to HPSCTerrSys/clm5_0
that referenced
this issue
Jun 27, 2024
…d96b0bc6cd6a302e187b0942c9462ad2cbb39' into eclm
adrifoster
pushed a commit
to adrifoster/CTSM
that referenced
this issue
Jul 18, 2024
…_006 Merge up to ctsm5.2.006 and deconflict
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Bill Sacks < sacks@ucar.edu > - 2012-03-22 16:20:18 -0600
Bugzilla Id: 1483
Bugzilla CC: rfisher@ucar.edu,
There are lots of checks in the code that look like this:
if (ltype(l) == istsoil .or. ltype(l) == istcrop)
or:
if (ltype(l)==istice .or. ltype(l)==istice_mec) then
or:
elseif ( itypelun(l) == istdlak .or. itypelun(l) == istslak ) then
I would propose adding some logical variables like:
is_vegetated(l) (true for soil & crop landunits)
is_ice(l) (true for ice & ice_mec landunits)
is_lake(l) (true for deep & shallow lake)
This would have these advantages:
more easily extensible: e.g., could add a new vegetated landunit without having to change conditionals that occur throughout the code
safer: e.g., I just found an instance where a conditional checked ltype(l)==istice, forgetting to also check the possibility of istice_mec
The text was updated successfully, but these errors were encountered: