-
Notifications
You must be signed in to change notification settings - Fork 11
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
MPRester #18
Comments
Hi, We might need to make corrections to adapt the revised mpi of pymatgen. |
I don't have the same error. Could you please install mp-api with the following command, which would solve the first issue? |
Thank you for your reply. I have install mp-api via
|
Before testing, I upgraded pymatgen and mp-api. If the following script doesn't work, the issue is not related to vise.
|
Hello,
I am currently learning how to use Vise by following the tutorial at https://kumagai-group.github.io/vise/.
It seems that the MPRester module has changed location in recent pymatgen versions, leading to an error.
This output is from pytmatgen 2024.11.13.
Replacing this import statement by
from pymatgen.ext.matproj import MPRester
may partially solve this error. However, simply replacing it yields another error:This may be a very basic question, although I could not figure out how to proceed further so far.
I would appreciate if someone could tell me how to solve this issue.
Thank you very much.
The text was updated successfully, but these errors were encountered: