Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add Google ARCO ERA5 data #132

Merged
merged 6 commits into from
Jul 2, 2024
Merged

Add Google ARCO ERA5 data #132

merged 6 commits into from
Jul 2, 2024

Conversation

jbusecke
Copy link
Contributor

No description provided.

@jbusecke
Copy link
Contributor Author

I just got this error and am a bit confused by it. I assume it has something to do with the fact that I removed the recipe object in meta.yaml here (this is supposed to simply link an existing arco dataset.

Secondly could we have caught this on the repo CI checks @andersy005? Could you explain what is different between 'validate-and-generate' and 'validate' which ran successfully on the feedstock?

@norlandrhagen
Copy link
Contributor

I think this lint PR leap-stc/arco-era5_feedstock#3 to the yaml should fix the catalog validation error.

@norlandrhagen
Copy link
Contributor

I think this should be G2G @jbusecke

@andersy005
Copy link
Collaborator

Secondly could we have caught this on the repo CI checks @andersy005? Could you explain what is different between 'validate-and-generate' and 'validate' which ran successfully on the feedstock?

i just checked, and there are some small differences in the validation script when validating a single feedstock (which is what happens in validate) versus a list of feedstocks (in validate-and-generate). i can look into merging both approaches in a consistent manner at a latter time.

@norlandrhagen norlandrhagen merged commit d015888 into main Jul 2, 2024
4 checks passed
@norlandrhagen norlandrhagen deleted the add-arco-era5 branch July 2, 2024 15:50
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants