ChemEnv integration about emmet HOT 14 CLOSED

@munrojm Awesome! 🥳 Thank you so much!

from emmet.

@JaGeo, feel free to try pulling the data with the latest MPRester and let me know what other changes you would like to make. We are going to be working on website integration soon.

from emmet.

I think I will just make a pull request for the option where we only do the analysis when oxidation states are available. I will comment the other code for now. It runs at least twice as fast and should also be easier to understand for users if only one type of ChemEnv analysis is used.

from emmet.

@JaGeo, just pinging you here to let you know I am debugging some builder pipeline issues which is delaying me on getting the chemenv stage up and running. This should be done soon, and I will update this thread.

from emmet.

@munrojm Thanks for the update! 🙂

from emmet.

@JaGeo, the builder runs well! I am able to get it completed in ~5hrs within our build pipelines. The data should be incorporated into the DB and available through the API next release.

from emmet.

@munrojm If there is more documentation needed to make this available, just let me know. I am happy to help with this as well.

from emmet.

@JaGeo, I appreciate the offer. Apologies again for the delay on this. This is essentially ready to go on the API side. Our initial intent was to wait to bundle this with some other data in the next DB release, but that kept getting pushed back for a number of reasons. Currently, I am working to fix up thermo data related issues. We intend to deploy again soon with to accommodate the changes. What I will do is include the ChemEnv data in that deployment so it is available next week.

from emmet.

Great, thank you!

from emmet.

I do have an additional question: would it be in principle possible to get a DOI for such a dataset (the whole one)?

from emmet.

@JaGeo The DOIs we get from OSTI need to be matched to a landing page. Dataset contributions to core and not MPContribs don't have that, and thus we can't assign DOIs for them. @tschaume can weigh in on this in case I am getting something wrong.

On another note, do you have any particular things you would like to have included as filters on the chemenv data within the API? chemenv_iucr for example?

from emmet.

@munrojm I think a filter on the environment names would be good: chemenv_iucr, chemenv_name, chemenv_iupac and maybe also a filter on the "continous symmetry measure" (csm) to exclude very distorted environments,, for example. We also already show some outputs from ChemEnv on the materialsproject toolkit page. There, we also display wyckoff_positions, and species and the name chemenv_name.

from emmet.

Thank you, @munrojm ! I will try to test it during this week!

from emmet.

@munrojm, I was too curious and I added some suggestions here: materialsproject/api#771

from emmet.