Skip to content

potcar spec kwarg #83

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

Merged
merged 2 commits into from
Mar 9, 2022
Merged

potcar spec kwarg #83

merged 2 commits into from
Mar 9, 2022

Conversation

jmmshn
Copy link
Contributor

@jmmshn jmmshn commented Mar 9, 2022

Unused potcar_spec kwarg

It looks like there is an potcar_spec kwarg here

potcar_spec: bool = False,

which might be useful for live debugging, but write_vasp_input_set cannot use it since it breaks the input set creation here:
vis = input_set_generator.get_input_set(structure, prev_dir=prev_dir)

This should allow you to write input files but it still looks like the KPOINTS file is missing.

Is this the expected behavior?

from atomate2.vasp.sets.core import RelaxSetGenerator
from atomate2.vasp.files import write_vasp_input_set
from pymatgen.core import Structure

mgo_structure = Structure(
    lattice=[[0, 2.13, 2.13], [2.13, 0, 2.13], [2.13, 2.13, 0]],
    species=["Mg", "O"],
    coords=[[0, 0, 0], [0.5, 0.5, 0.5]],
)
gen = RelaxSetGenerator()
write_vasp_input_set(mgo_structure, input_set_generator=gen, potcar_spec=True)

### creates inputfile but missing KPOINTS

@jmmshn
Copy link
Contributor Author

jmmshn commented Mar 9, 2022

Oh nvm I was not aware of this VASP flag
https://www.vasp.at/wiki/index.php/KSPACING

@utf
Copy link
Member

utf commented Mar 9, 2022

Thanks! Good spot.

@utf utf merged commit fa71fb8 into materialsproject:main Mar 9, 2022
@utf
Copy link
Member

utf commented Mar 9, 2022

I just noticed you didn't add docstrings for the new kwargs. Would you be able to add them in please?

@jmmshn jmmshn mentioned this pull request Mar 9, 2022
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

Successfully merging this pull request may close these issues.

2 participants