Add CUEQ Parameterization to test_benchmark.py#1184
Add CUEQ Parameterization to test_benchmark.py#1184ThomasWarford wants to merge 6 commits intoACEsuit:developfrom
Conversation
b890f93 to
6e1e502
Compare
|
Could you share a stacktrace for one of the failing compile test cases? I thought this was working and can you double check if you have the latest develop branch (#1170 is the PR you want) |
|
I realised this was unclear: you will need PR #1175 as well to combine cueq with compile but wanted to check if you are seeing compilation fail without cueq? |
|
The error persists with the latest commits from develop. The errors are only for |
|
I can wait on that PR before investigating further, if that would make sense |
|
The error above is because MACE-MP-0 doesn't have the required keys. Changing the loaded model to MACE-MP-0b3 seems to fix things. |
|
Even with Should this work @ilyes319 ? Can dig deeper if you think it's worthwhile |
|
cueq rewrites the model in ways that means the on-disk format needs to match the settings of the model you are loading. A possible workaround would be to load the model then convert to cueq by calling this: https://github.com/ACEsuit/mace/blob/main/mace/cli/convert_e3nn_cueq.py |

Builds upon some of the tests made by @hatemhelal - a quick way to gauge CUEQ speedups.
Before adding these changes, the tests with compile_mode='default' failed. The tests with compile_mode='default' still fail.
The tests with size-9 fail due to OOM errors.
Successful results: