-
Notifications
You must be signed in to change notification settings - Fork 3
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
consider kwarg for mapping noble gases to a "nearby" element or (better) using different elemental featurizer #82
Comments
I do not know if you already investigated this, but sometimes it tends to help to not encode atomic numbers but something that is closely related to chemistry (and varying with a more consistent trend across the PTE) such as the electronegativity. Maybe one could give the user the option to specify any If you want me to make a PR for that, simply assign me to this issue. |
That would be great! The modified Pettifor scale is one that I considered, mostly based on prior experience with ElMD (see e.g. |
Since I ended up writing the same code in different packages, I started collecting it in |
@kjappelbaum is |
I think so - I use it already in one of my other projects. I was waiting for the |
Ok, cool! Not sure if you saw, Thank you! I appreciate you getting involved despite many other priorities I'm sure |
Awesome, looking forward to seeing how this impacts |
@kjappelbaum this weekend I'm planning to try doing a distributed hyperparameter optimization on my uni's HPC via RayTune. Getting RayTune and Slurm to play nicely together might take some time (my first time connecting the two), but my goal is to at least get a dummy run operating. Two hyperparameters that I think would be really important to have for the full run (which might not happen until next week) are the elemental encodings and lexicographic site sorting (#75). |
ok, the PR is anyhow close to being done. I'll finish it tomorrow morning |
But also looking forward to seeing the RayTune + Slurm code 👍🏽 |
Awesome, I just took a closer look at the changes and am seeing that now. Thank you!
Sounds good! I hope I don't disappoint haha |
e.g.$He \rightarrow H\space\mathrm{or}\space Li$
Issue with generated structures having noble gases might be helped by:
The text was updated successfully, but these errors were encountered: