-
Notifications
You must be signed in to change notification settings - Fork 32
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
import MPRester
got TypeError: issubclass() arg 1 must be a class
#809
Comments
Based on the stack trace, it looks like this error originated in |
@njzjz Forgot to ask earlier: Could you retry this with Python 3.10+ to see if the error disappears? |
I tried Python 3.8, 3.9, and 3.10. The error only appears with Python 3.8. |
I confirm that it is a workaround to install an old version of pip install 'mp-api<0.33; python_version < "3.9"' |
@munrojm Maybe the min Python on |
Sorry for the delayed response on this. Yes, it looks like that may have to happen. |
I got the same error from "from mp_api.client import MPRester" with Python 3.9.16. My mp-api version is 0.33.3. |
The upstream package has a bug in Python 3.8 (materialsproject/api#809), so we decide to drop Python 3.8 support.
The upstream package has a bug in Python 3.8 (materialsproject/api#809), so we decide to drop Python 3.8 support. --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
you can try upgrade package typing-extensions |
Describe the bug
When
from pymatgen.ext.matproj import MPRester
, the following errors occur:To Reproduce
It can be reproduced in a clean Python 3.8 image.
The following errors occur:
Expected behavior
No errors.
Screenshots
Environment (please supply relevant versions and platform info):
Additional context
This issue might be related:
pydantic/pydantic#1298
The text was updated successfully, but these errors were encountered: