MetOffice / CSET

Toolkit for evaluation and investigation of numerical models for weather and climate applications.
https://metoffice.github.io/CSET/
Apache License 2.0
8 stars 1 forks source link

Include cartopy downloaded resources in GitHub Actions cache #647

Closed jfrost-mo closed 1 month ago

jfrost-mo commented 1 month ago

Fixes #646

Contribution checklist

Aim to have all relevant checks ticked off before merging. See the developer's guide for more detail.

github-actions[bot] commented 1 month ago

Coverage

jfrost-mo commented 1 month ago

I'm still a little concerned that there is a race condition if the documentation build finishes first, as then the cartopy resources won't be downloaded. Its probably fine though, as then we just fall back to what we had.

jfrost-mo commented 1 month ago

Eh, its probably fine. Worst case we can separate the docs cache from the other cache, but that would slow things down a bit.

jfrost-mo commented 1 month ago

Merging without review as this is a performance GH Actions tweak, and has no impact on CSET code.