-
Notifications
You must be signed in to change notification settings - Fork 187
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
Remove e3nn
version pin
#555
Comments
I tested it, and we seem to support the latest e3nn version. This will not be an easy decision, as it could weaken code interoperability, especially given how poorly TorchScript behaves. In which context is this problematic for you? |
In my personal opinion, interoperability should be dependent on the individual packages being used. So, if For me, it is not a huge issue. I package several machine learning workflows in my code quacc, which includes but is not limited to MACE. I allow users to Edit: It was @zulissimeta who originally reported this to me. |
This is a feature request to ultimately remove the version pin of
e3nn==0.4.4
insetup.cfg
to allow for more flexible versions (if possible).The text was updated successfully, but these errors were encountered: