Alex Ganose comments

Results 172 comments of


                                            Alex Ganose

Support for autosummary :recursive: documentation of entire API

Hi, I'm just wondering if there's any update on this issue?

Citing quadpy

Thanks for the pointer to the zenodo repository.

sumo-bandplot error: KeyError: 'sumo.plotting/orbital_colours.conf'

What is the command you are using to plot?

Hi all, I want to plot all the orbitals associated with my materials in the band structure. How to do this?

Unfortunately the projection colours are hardcoded here: https://github.com/SMTG-UCL/sumo/blob/f754c45dbd2f387c8fb0d2dd06c991fdc4f9be0f/sumo/plotting/bs_plotter.py#L627 It would be good to make this customisable.

DOS plot problem for a specified element, plots orbital instead

This is a missing feature which we definitely should support (it is often one of the most common ways of presenting density of states/projected band structures).

Does SUMO support plotting projection band structures according to lm_orbiltals?

It should be simple to add the function you suggest to Pymatgen via either method. I prefer the modifying the existing function tbh. We could then look at implementing this...

Does SUMO support plotting projection band structures according to lm_orbiltals?

Hi @zrqustc, this is correct. But the orientation of the spherical harmonics is determined by the cartesian x, y, and z directions. If bonds between atoms do not fall exactly...

Does SUMO support plotting projection band structures according to lm_orbiltals?

I made a rough diagram indicating what I mean: ![Screen Shot 2019-12-03 at 11 03 02 AM](https://user-images.githubusercontent.com/1330638/70081015-89505680-15bc-11ea-9281-eb1ed8db83a3.png) Note this issue will also affect lm-decomposed density of states...

ability to colour bands based on eigendisplacements in `phonon-bandplot`

@badw I know you have a code that can do this. Any interest in combining it into sumo?

Brillouin zone plotter

Thanks for this, implementation looks nice and clean! Main comment is can you make this pep8 compatible (i.e. 4 spaces for an indent, line length not longer than 80 characters)....