Skip to content
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

Open
billsacks opened this issue Dec 16, 2017 · 0 comments
Open

add meta-types for landunit types #5

billsacks opened this issue Dec 16, 2017 · 0 comments
Labels
code health improving internal code structure to make easier to maintain (sustainability)

Comments

@billsacks
Copy link
Member

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

@billsacks billsacks added this to the future milestone Dec 16, 2017
@billsacks 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
@billsacks 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 billsacks removed this from the future milestone Nov 5, 2018
@billsacks billsacks removed the priority: low Background task that doesn't need to be done right away. label Jan 16, 2019
@ekluzek ekluzek mentioned this issue Aug 21, 2019
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
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
glemieux referenced this issue in glemieux/ctsm Jun 25, 2021
billsacks pushed a commit to billsacks/ctsm that referenced this issue Jul 11, 2022
slevis-lmwg referenced this issue in slevis-lmwg/ctsm Aug 1, 2023
Fix testlist so options are under <test> rather than <machines>.
wwieder added a commit to wwieder/ctsm that referenced this issue Apr 9, 2024
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
Labels
code health improving internal code structure to make easier to maintain (sustainability)
Projects
None yet
Development

No branches or pull requests

1 participant