-
Notifications
You must be signed in to change notification settings - Fork 11
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
Deprecate CalcInterface #250
Conversation
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.
Indeed this does seem pretty straightforward! It looks like you just need to merge with the latest master branch and you should be good.
Edit: obviously you're well aware of the need to merge and as you note above, of course it's the develop branch in our case :)
aospy/automate.py
Outdated
@@ -237,7 +236,7 @@ def _combine_core_aux_specs(self): | |||
|
|||
def create_calcs(self): | |||
"""Generate a Calc object for each requested parameter combination.""" | |||
return [Calc(CalcInterface(**sp)) for sp in | |||
return [Calc(**sp) for sp in |
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.
Could this fit on one line now?
Very weird, this test is passing on my local machine... |
Disregard previous comment; hadn't pushed latest changes. In it goes! |
C.f. #249. Turns out this was easier than I thought it would be.
Although I accidentally forgot to pull the latest commits to my local develop branch before creating this branch, so I'll have to resolve the merge conflicts manually.
Given how easy this was, I'm inclined to proceed with it.