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

Update history string objects #316

Merged
merged 8 commits into from
Jul 12, 2024
Merged

Conversation

justin-richling
Copy link
Collaborator

Clean up the history string objects in adf_info.py:

  • clean up naming of history string variables

  • Make baseline history info more robust; set default if not provided

  • Give full dictionary of test and baseline case(s) history string lists to adf object

Update tape_recorder.py for new history string format:

  • Add check in history string lists for h0 time series files

Closes #315

@justin-richling justin-richling requested a review from nusbaume July 1, 2024 19:05
Copy link
Collaborator

@nusbaume nusbaume 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, but I did have a few questions and (hopefully minor) change requests.

lib/adf_info.py Outdated Show resolved Hide resolved
scripts/plotting/tape_recorder.py Outdated Show resolved Hide resolved
scripts/plotting/tape_recorder.py Outdated Show resolved Hide resolved
scripts/plotting/tape_recorder.py Outdated Show resolved Hide resolved
The ADF was missing the correct call for baseline history strings, this fix will make sure to grab the correct strings.
@justin-richling
Copy link
Collaborator Author

@nusbaume I got your suggestions done, but it looks like we still have some issues with the new history string implementation; the baseline history strings were not being brought over from the yaml file correctly.

I have a quick addition in adf_diag that should take care of this bug, so if you wouldn't mind taking another quick look, thanks!

@justin-richling justin-richling added the config files Concerns the ADF config (YAML) files label Jul 10, 2024
Copy link
Collaborator

@nusbaume nusbaume left a comment

Choose a reason for hiding this comment

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

Everything looks great now, thanks @justin-richling!

justin-richling added a commit to justin-richling/ADF that referenced this pull request Jul 11, 2024
Bring in changes from PR NCAR#316

Update time fix for old CESM time series files
@justin-richling justin-richling mentioned this pull request Jul 11, 2024
@justin-richling justin-richling merged commit 0e6e19c into NCAR:main Jul 12, 2024
7 checks passed
@justin-richling justin-richling deleted the pr-fix branch July 12, 2024 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
config files Concerns the ADF config (YAML) files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tape Recorder plots failing after recent history string update
2 participants