-
Notifications
You must be signed in to change notification settings - Fork 467
Remove built-in ADC module #2737
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
Conversation
For methods (or methods in certain circumstances, say reference or conv/df) only available through an external add-on, do we want those opt-in? That is, certainly the external must be (1) installed and detectable. But do we also want to (2) require the user to I can go either way, and I guess I'm now leaning toward not requiring (2) and adjusting MRCC accordingly. But it seems like something to discuss and settle on a consistent treatment. |
I think in the case of ADC, it makes little sense to require an explicit |
@loriab I removed the |
My best guess at why fnocc is using MEMORY is so that it can get more memory under default conditions (1gb instead of 0.5gb). Better handling would probably be to have a |
Please mark this PR as closing #1033. |
Will |
I can do that, sure 👍🏻 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hooray!
Remember to move the ADC code (including the equation PDF!) to the attic. @loriab will need to file that fnocc
issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Description
This PR removes Psi's internal ADC module, so that all ADC calculations are run through
adcc
from now on.Closes #1033.
User API & Changelog headlines
Dev notes & details
proc.py
Questions
Checklist
Status