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
All the Potentials are not written properly #3695
Comments
@aakhiltayal to be clear, this only occurs when there's a single absorbing atom in the unit cell, correct? |
@matthewcarbone No it occurs even for two atom system as I shown for FeN. When I pull all the structure for FeN, for some of structures all the potentials are written. I think it is because they have composition like Fe2N2 and have four atom in the structure but for cases where composition is like Fe1N1 and only two atom in structure it did not write all the potentials. |
No I mean a single absorbing atom in the unit cell. Based on the structure, it looks like there's a single Fe/unit cell. I don't think this problem occurs when there are multiple, even symmetrically equivalent, atoms in the unit cell. And just to be clear, what I mean by this is that this occurs in the case when you have some absorbing atom type and there is exactly one of those atoms in the unit cell. |
@matthewcarbone That is true |
pymatgen/pymatgen/io/feff/inputs.py
Line 927 in ac8a7e9
@matthewcarbone
While using the Potentials card to write potential it was found that it did not write all the potentials in the feff.inp file which makes the feff.inp file not usable for the further calculations. The exclusion of L927-L928 fix the issue but needs to be checked if it didn't create other issues as it was put deliberately.
for e.g the structure is pull from the materials project and passes to the Potential card the output doesnot have all the potentials but if L927 and L928 is excluded the output is correct.
The text was updated successfully, but these errors were encountered: