Skip to content

mean clim json catalogue read in #1307

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

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Conversation

lee1043
Copy link
Contributor

@lee1043 lee1043 commented Jul 9, 2025

In case default is not specified in catalogue json (e.g., as in the new ref clim dataset catalogue json file) and only one dataset is included in the json per variable, consider it as default.

This PR replaces #1306

…et included in the json, consider it as default
@lee1043 lee1043 mentioned this pull request Jul 9, 2025
@lee1043 lee1043 marked this pull request as ready for review July 16, 2025 19:50
@lee1043 lee1043 self-assigned this Jul 16, 2025
@lee1043 lee1043 linked an issue Jul 16, 2025 that may be closed by this pull request
@lee1043 lee1043 added this to the 3.9.2 milestone Jul 16, 2025
@lee1043 lee1043 requested a review from kristinchang3 July 16, 2025 19:51
@lee1043
Copy link
Contributor Author

lee1043 commented Jul 16, 2025

I think this PR is also ready to review. @kristinchang3 could you review when you have time?

Reviewing will need running demo 0, demo1a, and demo 1b notebooks after installing PMP from this branch.

Just for a friendly reminder: instruction for reviewing PMP PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: ref_dataset_name in mean_climate_driver.py
1 participant