Skip to content

generate_gdds: Get file lists earlier#3738

Merged
samsrabin merged 10 commits into
ESCOMP:b4b-devfrom
samsrabin:generate_gdds-get-file-lists-earlier
Feb 5, 2026
Merged

generate_gdds: Get file lists earlier#3738
samsrabin merged 10 commits into
ESCOMP:b4b-devfrom
samsrabin:generate_gdds-get-file-lists-earlier

Conversation

@samsrabin
Copy link
Copy Markdown
Member

@samsrabin samsrabin commented Feb 4, 2026

Description of changes

Refactors generate_gdds and its helper module so that file lists are generated early in the process. This makes it so that, if there are any files missing, errors will be thrown quickly.

It also adds a lot of testing.

Specific notes

Contributors other than yourself, if any: None

CTSM Issues Fixed: None

Are answers expected to change (and if so in what way)? No

Any User Interface Changes (namelist or namelist defaults changes)? No

Does this create a need to change or add documentation? Did you do so? No

Testing performed, if any: Tested as part of my crop reparameterization work.

  • Python unit tests
  • Python system tests
  • clm_pymods suite
  • rxcropmaturity suite

@samsrabin samsrabin self-assigned this Feb 4, 2026
@samsrabin samsrabin added b4b bit-for-bit PR status: awaiting review Work on this PR is paused while waiting for review. PR status: needs testing test: python Pass clm_pymods test suite plus Python sys/unit tests before merging labels Feb 4, 2026
@samsrabin samsrabin marked this pull request as draft February 4, 2026 22:41
Comment thread python/ctsm/test/test_unit_generate_gdds.py
Comment thread python/ctsm/test/test_unit_generate_gdds.py
Comment thread python/ctsm/test/test_unit_generate_gdds.py
@samsrabin samsrabin marked this pull request as ready for review February 4, 2026 23:14
Copy link
Copy Markdown
Contributor

@slevis-lmwg slevis-lmwg left a comment

Choose a reason for hiding this comment

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

@samsrabin I browsed through this, and I'm approving. Pls let me know if there are parts that you would like me to look at carefully or that you would like to discuss in a meeting.

Comment thread python/ctsm/crop_calendars/generate_gdds.py
@samsrabin samsrabin removed the PR status: awaiting review Work on this PR is paused while waiting for review. label Feb 5, 2026
@github-project-automation github-project-automation Bot moved this to Ready to start (or start again) in CTSM: Upcoming tags Feb 5, 2026
@samsrabin samsrabin moved this from Ready to start (or start again) to In progress - b4b-dev in CTSM: Upcoming tags Feb 5, 2026
@github-project-automation github-project-automation Bot moved this from In progress - b4b-dev to In progress - master in CTSM: Upcoming tags Feb 5, 2026
@samsrabin samsrabin moved this from In progress - master to In progress - b4b-dev in CTSM: Upcoming tags Feb 5, 2026
@samsrabin samsrabin merged commit 5bd712f into ESCOMP:b4b-dev Feb 5, 2026
6 checks passed
@github-project-automation github-project-automation Bot moved this from In progress - b4b-dev to Done (non release/external) in CTSM: Upcoming tags Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

b4b bit-for-bit PR status: needs testing test: python Pass clm_pymods test suite plus Python sys/unit tests before merging

Projects

Status: Done (non release/external)

Development

Successfully merging this pull request may close these issues.

2 participants