-
Notifications
You must be signed in to change notification settings - Fork 198
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
Loosen the dependency to avoid conflict with other package #588
Comments
Can you try locally updating to the latest e3nn and see if everything training works. |
I am trying to wrap a pyproject.toml dependency. |
I meant can you confirm that locally e3nn latest version works fine? |
Not sure at this moment. I could update locally and try training for the next model. Another way is test workflow? Is there anything missing in the test that we should probably add? |
big missing part in the testing workflow atm are training with h5 files and multigpu (that one we can't not really test). |
I see. I will try adding offline testing script and small demo data. And pick up #452 and solve it altogether |
related to #555 |
I figured something changed in the Clebsch Gordan convention between 0.4.4 and 0.5.1 that breaks backward compatibility with foundation models so I will keep the pin for e3nn for now. |
@ilyes319 could we loosen the dependency of e3nn to
e3nn>=0.4.4
mace/setup.cfg
Line 18 in 22a2e3e
The text was updated successfully, but these errors were encountered: