-
Notifications
You must be signed in to change notification settings - Fork 20
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
Remove the dependence on vise.yaml in make_efnv_correction #19
Comments
Thank you for contacting me. |
Unfortunately, DefaultsBase cannot be removed for the command line tools. |
I upgrade pydefect to 0.9.4. Please try it. |
Thank you for your quick response. In my case, I do not have vise.yaml and pydefect.yaml used for VASP calculations. For my purpose, I make the following naive modification in pydefect/defaults.py: #from vise.defaults import DefaultsBase @singleton
... With the above change my python script runs and give expected results. As you commented out, the above change will not be good for pydefect as a whole. |
I am very interested in pydefect and I make some tests following the implementation in test_kumagai as in pymatgen.analysis.defects:
https://github.com/materialsproject/pymatgen-analysis-defects/blob/main/tests/test_corrections.py
(the test_kumagai part is presently commented out)
Unfortunately, my tests are always running forever and never end. If terminated forcibly, the process will return a message with vise.yaml not found, and also other messages regarding the defaults.
In my case, I make my own script to construct CalcResult, not from VASP output. If structure, charge, dielectric tensors, and atom-site potentials are available, it is desired that vise.yaml is not necessary anymore. Thank you very much.
*vise and pydefect are installed before my tests.
pip install vise
pip install pydefect
The text was updated successfully, but these errors were encountered: