You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'd like to improve the syntax for inheriting VASP settings and remove some boiler plate code in sub-flows. Rather than having devs use incar.copy() + incar.update(), we can replace those with a simple incar_updates attribute that is set instead. incar with then be a property method that builds the incar (and caches it for speedup).
I'd like to improve the syntax for inheriting VASP settings and remove some boiler plate code in sub-flows. Rather than having devs use
incar.copy()
+incar.update()
, we can replace those with a simpleincar_updates
attribute that is set instead.incar
with then be a property method that builds theincar
(and caches it for speedup).current approach
new approach
@SWeav02 let me know what you think of the proposed update. I'll wait on your approval before changing this throughout the repo
The text was updated successfully, but these errors were encountered: