-
Notifications
You must be signed in to change notification settings - Fork 882
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
Simplify case handling in lobster.inputs
#3848
Conversation
lobster.inputs
lobster.inputs
Do we have further concerns? @JaGeo @QuantumChemist If not I might ping Janosh for reviewing? |
@DanielYang59 I don't have time at the moment. I would really appreciate that we merge this very carefully to avoid any breakages - especially also of Lobster workflows in atomate2. |
Sure no rush at all. Let's leave it until you have time to review :) |
Thanks @DanielYang59 for understanding. I really appreciate your work here but cannot keep up with the pace. Sorry! |
All good, I'm not in a rush at all :) Just tagging it to make everything organized. Thanks! |
I don't see any more things that require changes. I could check if it breaks LobsterPy's functionality over the weekend (this or the next ones). Sorry, I'm also too busy to test the changes in depth atm. |
No worries, that would be very helpful too :) |
(I am working on it - it just takes a while. Afterwards, it is good to merge from my side.) |
It's fine. The Lobster tests in atomate2 and LobsterPy tests are all passing. From my side, it is ready to merge. |
Thanks for the test :) I would ping @janosh for final review. |
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, thanks @DanielYang59! 👍
Simplify case handling in
lobster.inputs
LISTKEYWORDS
toLIST_KEYWORDS
for consistency.Use
{lowercase: camalCase}
to simplify key case handling, for the following methods:Now lower-case keys are used internally, and Camel case keys are reserved in case of writing (good looking) output files.
dict
methodsfrom_file
write_lobsterin
: Also use it to recover the key case.diff
TODOs
lobsterin
keywords