-
Notifications
You must be signed in to change notification settings - Fork 12
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
Replace repulsion restraint with ZBL reference #234
Comments
Do you have a preferred way of implementing this? I like the idea of allowing for a general "reference potential" that is subtracted before the fit, which might be ZBL, the one-body E0s, or some combination. But might also be a better way. |
The possibility is already implemented. Just need to supply a convenient interface for it. I don't have a strong view on this yet. |
Sure, I was asking about the interface and your opinion - fine if you don't have a strong view. I saw a change somewhere recently that moved in the direction of E0s as the only reference and away from the more generic approach. Wasn't sure if that was purposeful or just convenient in the moment. |
Think the E0s (list of pairs or Dict) and ZBL (true/false)should have convenience kwargs but can be overwritten by passing a general Vref. |
For now at least. Over time we can think about how to improve on this. |
I've now translated the repulsion restraint (#262). Still want to add the ZBL option so we can compare the two. |
addressed by #263 - I'm not 100% happy with the tests, but I think it now needs to be tried on real systems before raising issues with the implementatin. |
No description provided.
The text was updated successfully, but these errors were encountered: