Skip to content

fix nbp time range and add final metrics#33

Closed
nocollier wants to merge 7 commits into
mainfrom
final-ilamb-additions
Closed

fix nbp time range and add final metrics#33
nocollier wants to merge 7 commits into
mainfrom
final-ilamb-additions

Conversation

@nocollier
Copy link
Copy Markdown
Contributor

Description

Checklist

Please confirm that this pull request has done the following:

  • Data registry up to date (regenerate if necessary with a comment on this PR of /regenerate)
  • Documentation added (where applicable)
  • Changelog item added to changelog/

@nocollier
Copy link
Copy Markdown
Contributor Author

/regenerate

@github-actions
Copy link
Copy Markdown

PR comment handling
Something went wrong!

Details: https://github.com/Climate-REF/ref-sample-data/actions/runs/14995962085

@nocollier
Copy link
Copy Markdown
Contributor Author

/regenerate

@github-actions
Copy link
Copy Markdown

PR comment handling
Something went wrong!

Details: https://github.com/Climate-REF/ref-sample-data/actions/runs/15001153802

@nocollier
Copy link
Copy Markdown
Contributor Author

nocollier commented May 13, 2025

I am assuming this is failing because the thetao is too large (time, depth, lat, lon) to download. It would help if I could add some time slicing logic as suggested in this issue. This isn't standard because the metadata records for datetime are often not correct.

@nocollier
Copy link
Copy Markdown
Contributor Author

/regenerate

@github-actions
Copy link
Copy Markdown

PR comment handling
The regenerate task is done!

You can find the workflow here:
https://github.com/Climate-REF/ref-sample-data/actions/runs/15067889991

@nocollier
Copy link
Copy Markdown
Contributor Author

/regenerate

@github-actions
Copy link
Copy Markdown

PR comment handling
The regenerate task is done!

You can find the workflow here:
https://github.com/Climate-REF/ref-sample-data/actions/runs/15068275886

@nocollier nocollier closed this May 16, 2025
@nocollier nocollier deleted the final-ilamb-additions branch May 16, 2025 12:39
@lewisjared
Copy link
Copy Markdown
Contributor

We are starting to push the limits with this CI pipeline. Perhaps we need something to just regenerate a given request. That should be an easy enough fix

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

Successfully merging this pull request may close these issues.

2 participants