-
Notifications
You must be signed in to change notification settings - Fork 29
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
cli invert axis, add get site all orbitals dos plot #149
Conversation
Ready for review |
Thank you! I have the issue that it always plots the total dos as well. Is this on purpose? |
I intentionally kept that. But I can remove this hard coding. I thought it would be nice to always see the total dos when using other options for comparisons |
I don't think it's good. I barely could see the orbital-resolved DOS for some of the test cases. |
Can we make it optional? Or switch it off? |
Sure, we can make it optional . Will make changes tomorrow and let you know when I finish it. |
Hi @JaGeo, now the dos plotter should not add total dos by default and only adds it when |
I think it is |
Closes #142
Hi @JaGeo ,
DOS plotting options have been added
--invertaxis
option can plot DOS now via cli in chemistry conventions--orbital all
in conjuction with site index for plotting dos, generates a plot with all available orbitals