search

lucidrains / En-transformer

Implementation of E(n)-Transformer, which incorporates attention mechanisms into Welling's E(n)-Equivariant Graph Neural Network
MIT License
208 stars 28 forks source link

Is this line required? #1

Closed aced125 closed 3 years ago

aced125 commented 3 years ago

https://github.com/lucidrains/En-transformer/blob/7247e258fab953b2a8b5a73b8dfdfb72910711f8/en_transformer/en_transformer.py#L159

Is this line required? Does line 157, two lines above, make this line redundant?

lucidrains commented 3 years ago

@aced125 oh oops, thanks for catching that! removed in the latest commit