-
Notifications
You must be signed in to change notification settings - Fork 69
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
el_refs serialized as strings #576
Comments
Yup! Will take a look at this. |
@janosh, this seems to stem from the fact that |
Also, |
Was I tagged by accident? |
@janosh No, this just relates to a pymatgen issue I was about to open. We can discuss outside of this thread. |
emmet/emmet-core/emmet/core/thermo.py
Lines 251 to 253 in 31a2262
When these
PhaseDiagramDocument
s are pulled from the DB and deserialized the keys forel_refs
are string but thepymatgen
code usesElement
the entries time. I can't find if a conversion occurred in emmet.@kim-jiyoon reported this. @munrojm and @mkhorton can you please have a look? This is to get the conversion documents on the new site.
The text was updated successfully, but these errors were encountered: