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

Cuequivariance dependency leads to unsupported Python versions #775

Open
thomas-rocke opened this issue Jan 10, 2025 · 2 comments
Open

Cuequivariance dependency leads to unsupported Python versions #775

thomas-rocke opened this issue Jan 10, 2025 · 2 comments

Comments

@thomas-rocke
Copy link

Cuequivariance uses Python 3.10 features, which conflicts with the claim that mace supports Python >= 3.7.

Cuequivariance was initially an optional dependency, but 5b2bd9a promoted it to a mandatory dependency.

@ilyes319
Copy link
Contributor

ilyes319 commented Jan 10, 2025

@thomas-rocke, thank you that's a bug, cueq should stay optional dependency for now. I will switch back. Note that the Python versions are all over the place any atm, I need to make a good estimate, 3.10 is the most safe to go for, regardless of cueq or not.

@thomas-rocke
Copy link
Author

I've been able to use Python 3.9.6 with mace@4c1ad89 (which is after the introduction of cuequivariance, but before it became mandatory).

My recommendation would be to follow the officially supported versions (https://devguide.python.org/versions/) - Python 3.9 is supported until the end of this year, but it may be more convenient to preempt the end of life and just stick with 3.10+ as you suggested.

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

No branches or pull requests

2 participants