Skip to content

Added functionality for importing daily ERA5 data#136

Merged
albinwwells merged 5 commits intodevfrom
issue-127-fix
Oct 7, 2025
Merged

Added functionality for importing daily ERA5 data#136
albinwwells merged 5 commits intodevfrom
issue-127-fix

Conversation

@albinwwells
Copy link
Contributor

No description provided.

@albinwwells albinwwells requested a review from btobers October 6, 2025 20:27
@albinwwells albinwwells linked an issue Oct 7, 2025 that may be closed by this pull request
@btobers
Copy link
Collaborator

btobers commented Oct 7, 2025

@albinwwells - this PR will fail the testing suite until PR #134 is merged into dev. I made a couple minor variable name changes that went into the notebooks too and your branch does not yet have those changes, hence why the tests fail. Once we close #134, we'll merge dev back into issue-127-fix and then this PR should pass the test suite.

Copy link
Collaborator

@btobers btobers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@albinwwells left a commit for you to review, otherwise approved.

@albinwwells albinwwells marked this pull request as ready for review October 7, 2025 15:35
@albinwwells albinwwells merged commit 5949dd4 into dev Oct 7, 2025
1 check passed
@albinwwells albinwwells deleted the issue-127-fix branch October 7, 2025 15:54
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.

Enable importing of daily climate data

2 participants