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

Proposed models in tutorial 08 don't have daily data #456

Open
Mayetea opened this issue Jan 18, 2023 · 0 comments
Open

Proposed models in tutorial 08 don't have daily data #456

Mayetea opened this issue Jan 18, 2023 · 0 comments

Comments

@Mayetea
Copy link
Contributor

Mayetea commented Jan 18, 2023

In the tutorial 08 where we fetch climate data from the pangeo dataset, we propose a list of models to try and see the difference. The problem here is that many of the proposed model, only have 'Amon' (monthly?) data and no daily ('day') data, but we only try to train our model with daily data. And so with the proposed models that have no daily data, the code crash when trying to select them. Here's the list of all models who can't work in this tutorial:

AWI-CM-1-1-MR,
BCC-CSM2-MR,
CESM2-WACCM,
CMCC-CM2-SR5,
EC-Earth3-Veg,
FDL-ESM4

I can remove them in a PR, if you think they should be removed. It can also be changed as daily first and fall back to monthly if no daily is found. Let me know!

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

No branches or pull requests

1 participant