Skip to content

ufs-dev PRs 349 343#1194

Merged
grantfirl merged 18 commits intoNCAR:mainfrom
scrasmussen:ufs-devpr-349-343
Feb 24, 2026
Merged

ufs-dev PRs 349 343#1194
grantfirl merged 18 commits intoNCAR:mainfrom
scrasmussen:ufs-devpr-349-343

Conversation

Clara Draper and others added 17 commits January 12, 2026 16:45
- Removed 790 lines of isolated comment markers and excessive blank lines
- Removed 3 unused epsilon variables (epsoro, epsplr, epsacn)
- Removed 2 unused namelist variables (qcmsk, znlst)
- Removed 3 unused subroutines (count, dayoyr, maxmin)
- Removed 31 lines with developer suffix comments (cbosu, cggg, landice, cjfe)
- Kept bitmap-related comments but removed developer identifiers
- Removed all inline comments from routine argument lines
- Removed 7 commented-out old argument lines
- Removed 9 lines of commented-out old code (print statements and calculations)

Total reduction: ~900 lines from original 8204 to 7275 lines
- Added explanatory comment about snow data grid determination (gaussian vs lat/lon)
- Removed TG3 MODS BEGIN/END markers
- Removed obsolete file format documentation comments
Create new scheme to get fields provided by CDEPS Inline
Modified 'dtfac' OGWD tendency limiter in drag_suite_psl to match that in drag_suite_run
Copy link
Collaborator

@mdtoyNOAA mdtoyNOAA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@grantfirl grantfirl mentioned this pull request Feb 13, 2026
@scrasmussen scrasmussen mentioned this pull request Feb 13, 2026
@grantfirl grantfirl merged commit f4c82f6 into NCAR:main Feb 24, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants