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

partial occupancies error #22

Closed
rightsuman opened this issue Dec 21, 2020 · 5 comments
Closed

partial occupancies error #22

rightsuman opened this issue Dec 21, 2020 · 5 comments

Comments

@rightsuman
Copy link

Hi Lucy,
I am currently using your code effmass for my research and have faced an error which I haven't been able to solve. I am not very versed with using python so pardon me for stupid questions.
So actually the error comes as "You have partial occupancy numbers in Data.occupancy ....... You should check that the attributes Data.VBM, Data.CBM and Data.fermi_energy are correct, and if not, set them manually"
I am using your jupyter notebook example. I can't really figure out where or how I can set them manually.
It would be great if you could give me any suggestion regarding this

@lucydot
Copy link
Owner

lucydot commented Dec 22, 2020

Hello,

I'm a little stumped as to why you are getting that error with the example provided in the repo as the example OUTCARs do not have partial occupancy. I'm assuming that you are also reading in the example data I have provided rather than providing your own?

I've just tried to replicate your error but am unable to. Please could you -

a) tell me how you downloaded effmass (straight from github? or via pip?)
b) tell me at what point you got this error - is it when you try to run the third cell in the jupyter notebook (effmas/paper/notebook.ipynb)?
c) confirm that the effmass/paper/data directory contains all of the relevant files (those that are used in the command that is giving the error).

  • Lucy

@rightsuman
Copy link
Author

Hello,
So actually you got my context a bit wrong. I was using your example jupyter notebook code for my own calculation of effective mass. This is where I am getting the error and want to know how these parameters can be changed. I have tried for other materials and its working fine where the band gap is a bit larger.
For your informatiion:
a) I downloaded straight from Github and installed thereafter
b) Getting the error on the third line where we import the data leaving out the first few lines for hybrid calculations.
c) I am able to run this on other files, so I think the program is well in place.
Thanks for your reply!

@lucydot
Copy link
Owner

lucydot commented Dec 26, 2020

Hi,

Ok that makes more sense. If there is a small band gap then it is possible there are partial occupancies and the code is correct to flag this up.

  • First, check to see the attributes data.VBM, data.CBM and data.fermi_energy are what you would expect for your system.

  • If not, you should be able to re-assign correct values directly. For example (using the example in the notebook), if you have:

    data = inputs.Data("./data/CdTe/HSE06_SoC/OUTCAR","./data/CdTe/HSE06_SoC/PROCAR", ignore=216)

    you could re-assign with e.g.

    data.VBM = 0.345

As you have mentioned that you are not well versed in Python, I'll just highlight that in this example Data and data are different. Data is the Python class, whilst data is an instance of the Data class that has been created. If you need to adjust the VBM, CBM values you need to re-assign the class instance attributes .

@lucydot
Copy link
Owner

lucydot commented Jan 26, 2021

Hello @rightsuman can I close this issue? Has it been resolved?

@lucydot
Copy link
Owner

lucydot commented Mar 8, 2021

Ok I'm closing this issue, feel free to re-open if problem not resolved

@lucydot lucydot closed this as completed Mar 8, 2021
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

2 participants