search

SMTG-Bham / sumo

Heavyweight plotting tools for ab initio calculations
https://smtg-bham.github.io/sumo/
MIT License
192 stars 78 forks source link

Vertical DOS Plot #17

Open shyamd opened 6 years ago

shyamd commented 6 years ago

Right now, I'm hackishly using _makedos to do this myself but it would be really nice if there was an option in get_plot to produce a vertical plot or a get_vertical_plot that had many of the same options as get_plot.

ajjackson commented 6 years ago

This seems a bit niche to have in the CLI, as it's hard to choose an appropriate default layout. However, it also appears to be more cumbersome than it "should" be through the Python API; you shouldn't have to access an internal function (indicated with leading underscore) to do this. Maybe if we refactor the DOS plotting to share more between the regular horizontal plot and the vertical plot appended to a band structure, then as a side-effect it could be easier to access a vertical plot through the Python API.