-
Notifications
You must be signed in to change notification settings - Fork 3
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
Update requirements #5
Open
jroettenbacher
wants to merge
11
commits into
igmk:main
Choose a base branch
from
jroettenbacher:update-halo-hamp
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The HAMP dataset is now available as a unified dataset. The notebooks have been adapted already. However, the table of contents does not reflect this change. This commit fixes this.
- Matplotlib causes errors on install with the pinned version. Allowing a higher version than needs numpy at a higher version as well. - simplification causes errors in the current pinned version. A newer version doesn't. - jupyter-book and jupyter-text also cause errors with the pinned versions
ac3airborne and dotenv added to the requirements
include dask[dataframe]
… fixes Removed the AC3 cloud credentials from each notebook and some grammar fixes - updated simplification package requirement - simplification requires numpy 1.24.4
jroettenbacher
changed the title
Update Halo HAMP dataset and update requirements
Update requirements
Dec 16, 2024
This PR is work in progress and not ready to be merged |
There is currently an error when trying to set the intake local cache via an environment variable (see [Github Issue](intake/intake#855)). This block ensures that there is a valid local storage set, even when not environment variable is set.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi all,
while doing some work on the project I noticed that the requirements are rather old. They are not compatible with python 3.12 anymore and so I started to update them bit by bit and resolved errors along the way.
Due to the update of jupyter-book I needed to update some cross references.
We could also think about moving from sphinx as a built engine to MyST as jupyter book will be moving to MyST with version 2 of jupyter book. See: https://executablebooks.org/en/latest/blog/2024-05-20-jupyter-book-myst/
The biggest downside I can see is that the nice sphinx theme of the book is not available as of now.
ToDos:
Best Johannes