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
Initial Design of the Composition/Structure/Missing_value-related Metafeatures for the Input Dataset #30
Conversation
return 1 | ||
elif element.is_alkali: | ||
return 4 | ||
elif element.is_alkaline: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm pretty sure pymatgen's Element or Specie has some of these properties already
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I just use pymatgen Element object to do this. I just want to assign a number to the species that are of interest for the elements. This can be further used to perform set() etc to get the element types present in the formula, such as [1, 1, 1] or [1, 1] will be all transitional metal, [2, 2, 9] or [2, 9] will be rare_earth_metal-nonmetal etc.
Thank you @Qi-max ! Please feel free to merge anytime you are done and tests passed. We will discuss all our changes tomorrow. |
OK, cool! @albalu |
To-do: