Skip to content
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

🐛[BUG]: Install does not work for Python 3.12 #150

Open
NickGeneva opened this issue Oct 28, 2024 · 4 comments
Open

🐛[BUG]: Install does not work for Python 3.12 #150

NickGeneva opened this issue Oct 28, 2024 · 4 comments
Assignees
Labels
0 - Blocked Cannot progress due to external reasons bug Something isn't working

Comments

@NickGeneva
Copy link
Collaborator

Version

main

On which installation method(s) does this occur?

Pip

Describe the issue

Some of the ecwf libs are being annoying with 3.12... need to probable shift to optional install or something of those to allow newer python versions

@NickGeneva NickGeneva added ? - Needs Triage Need team to review and classify bug Something isn't working and removed ? - Needs Triage Need team to review and classify labels Oct 28, 2024
@mehranion
Copy link

It sounds like some libraries in the ECWF (ECMWF - European Centre for Medium-Range Weather Forecasts) suite have compatibility issues with Python 3.12. This is not uncommon, as Python updates can sometimes introduce breaking changes or deprecate certain APIs, requiring library maintainers to make adjustments.

Possible Approaches:

Optional Installation:

You can modify your setup to make these ECWF libraries optional. This would mean only installing them when users specify a particular option (e.g., pip install mypackage[ecwf]), which would give users flexibility based on their Python version.
Compatibility Check for Python 3.12:

Add a Python version check in your installation requirements, marking certain ECWF dependencies as incompatible with 3.12 until the libraries are updated. This way, users on Python 3.12 won’t experience failed installs.
Alternative or Newer Dependencies:

Check if newer versions of these libraries support Python 3.12 or if there are compatible alternatives available.
Environment Isolation:

Since library updates might be in progress, you could also suggest users work with Python 3.11 for ECWF dependencies until support for Python 3.12 stabilizes.

@NickGeneva NickGeneva self-assigned this Nov 19, 2024
@NickGeneva
Copy link
Collaborator Author

Blocked by Modulus now

@NickGeneva
Copy link
Collaborator Author

Fixed in PR: NVIDIA/modulus#729

Leaving open until next version of modulus is released on pipe.

For 3.12 users in the mean time first install modulus from github then install earth2studio:

pip install modulus git+https://github.com/NVIDIA/modulus.git
pip install earth2studio

@NickGeneva NickGeneva added the 0 - Blocked Cannot progress due to external reasons label Dec 12, 2024
@NickGeneva
Copy link
Collaborator Author

Presently blocked by torch harmonics: NVIDIA/torch-harmonics#58

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 - Blocked Cannot progress due to external reasons bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants