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

[WIP] Implement FIRE minimizer as CustomIntegrator #102

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

jchodera
Copy link
Member

@jchodera jchodera commented Nov 3, 2015

This PR implements the FIRE minimizer (suggested in #85) from

Structural relaxation made simple.
PRL 97:170201, 2006
http://dx.doi.org/10.1103/PhysRevLett.97.170201
which uses a simple heuristic scheme that respects constraints.

@proteneer
Copy link

Interested to know if you guys are still working on this. I'd like to find a way to get rid of the conditional inertia check: P:=F.v < 0 to make it differentiable

@jchodera
Copy link
Member Author

I think @hannahbrucemacdonald and @jaimergp were looking into this last.

Can't you just use a soft tanh-like function that scales the timestep by a continuously-variable amount depending on the sign and value of the momentum?

@proteneer
Copy link

I think that's definitely something we can look into, i.e. replacing the if/then (aka. heaviside function) with a tanh heaviside function.

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