Skip to content
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

y-axis values in sumo-dos plot #199

Open
mmmkhan opened this issue Jan 30, 2023 · 2 comments
Open

y-axis values in sumo-dos plot #199

mmmkhan opened this issue Jan 30, 2023 · 2 comments

Comments

@mmmkhan
Copy link

mmmkhan commented Jan 30, 2023

Is there away to add y-axis values in sumo DOS plot? I understand it's arb units, but I still want to add them.

Thanks,

@ajjackson
Copy link
Member

Hi @mmmkhan ,

this option is not currently exposed to the user. However if you are willing to tweak the code, all you need to do is "comment out" this line by adding a # to the start without changing the indentation level:

https://github.com/SMTG-UCL/sumo/blob/8f4f76e4f3fb38b5569f023a6ad087288e9507f7/sumo/plotting/dos_plotter.py#L392

Out of interest, what y scale do you expect to find on a DOS? Do you expect the DOS to be normalised to 1, the number of atoms per unit cell, the number of calculated bands, or something else?

@MengYin-ink
Copy link

Dear @ajjackson ,

I also want to set the DOS values since I want to compare the DOS and peak of two different systems. Therefore, the absolute results of DOS are much easier for comparison.

I have commented out the line mentioned above, however, the figure still does not show the values on the y-axis.
Is there anything else I need to do except comment out that line?

Best Regards,
Meng

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants