-
In the symmetric contraction block, which implements Eq. 10 and 11 from the MACE paper, one of the key hyperparams is Based on my understanding, if we set this param to True, the contraction step will additionally consider the My questions are:
|
Beta Was this translation helpful? Give feedback.
Replies: 2 comments 2 replies
-
And a similar query would apply to the implementation of the skip connection, e.g. https://github.com/ACEsuit/mace/blob/main/mace/modules/blocks.py#L513 Does this line of code follow Eq. 12 of the paper? Is it something similar to the NequIP paper's residual connection: https://www.nature.com/articles/s41467-022-29939-5#Sec1
|
Beta Was this translation helpful? Give feedback.
-
Your understanding is completly correct! The equation to look at is actually Eq.11n You can see that the weights there have a z_i index. In the algo these are selected by the one hots as you mentionned. While it is true that you would have the flexibility already, the full formulation is actually more flexible (the other would be a sort of tensor decomposition of what we are doing). Empirically it also gives the best results, for no extra cost! |
Beta Was this translation helpful? Give feedback.
Your understanding is completly correct! The equation to look at is actually Eq.11n You can see that the weights there have a z_i index. In the algo these are selected by the one hots as you mentionned.
While it is true that you would have the flexibility already, the full formulation is actually more flexible (the other would be a sort of tensor decomposition of what we are doing). Empirically it also gives the best results, for no extra cost!