Skip to content
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

Basis sparsification #2

Open
cortner opened this issue Jan 14, 2022 · 3 comments
Open

Basis sparsification #2

cortner opened this issue Jan 14, 2022 · 3 comments
Labels

Comments

@cortner
Copy link
Member

cortner commented Jan 14, 2022

If we want to allow solvers that sparsity, then we must also then be able to reduce the basis size accordingly. This is currently getting implemented in ACE.jl, but we may wish to backport it to ACE1.jl.

@cortner
Copy link
Member Author

cortner commented Jan 15, 2022

@casv2 @davkovacs - thoughts? Useful? Important enough to give it a milestone?

@casv2
Copy link
Contributor

casv2 commented Jan 17, 2022

I think this would be very useful indeed, ideally we'd get a lot of freedom of adding/removing basis functions when fitting. It would allow use to really explore the entire ACE basis and guide us towards tailor-made bases for every fit. We've seen that often the fit revolves around just a fraction of the entire ACE basis so I think there's a lot of interesting stuff to be done here.

@davkovacs
Copy link
Contributor

As for me, I have entirely abandoned sparsification, but would be happy to re-explore if it actually removed the basis functions. Previously I saw that I could get the same accuracy with 80% less basis functions, but it didn’t translate into speed or memory saving and was adding an extra parameter, so I stopped using it.

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

No branches or pull requests

3 participants