Skip to content

Conversation

pokutta
Copy link
Member

@pokutta pokutta commented Feb 14, 2021

No description provided.

@pokutta
Copy link
Member Author

pokutta commented Feb 14, 2021

basically implements https://arxiv.org/pdf/2102.02029.pdf

-> not done yet. need to change theory also to be more useful

end

# build-in NEP here
if nep === true
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
if nep === true
if nep

just checking the boolean directly

@matbesancon
Copy link
Member

looking quickly at the paper, doesn't this need new LMOs?

@pokutta
Copy link
Member Author

pokutta commented Feb 14, 2021 via email

@matbesancon matbesancon mentioned this pull request Feb 24, 2021
@matbesancon
Copy link
Member

matbesancon commented Mar 17, 2021

@pokutta I think you have merged this into another branch that got merged into master, it is already there - it is not quite ready because of the interfacing as the gradient change depends on the iteration. i will fix it for vanilla FW today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants