-
Notifications
You must be signed in to change notification settings - Fork 874
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
Add F* diagram generator to pymatgen #3277
base: master
Are you sure you want to change the base?
Conversation
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.
Thanks @jonathanjdenney!
Can you please check if some of the test files we already have can be used in place of new ones and for the remaining new files, please gzip
them.
compressed the cifs into fstar.tar.gz. Implemented type hints. reworked to simplify and run better.
remove else after continue
@janosh |
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.
As I mentioned earlier, please try to minimize the number of new test files and compress the remaining.
@janosh |
@jonathanjdenney There are still 33 changed files and 23k changed lines which make this PR very hard to review. The purpose of these 3 files is obvious. Could you either explain the purpose of or remove the other 30 files? |
@jonathanjdenney I removed all the excess files and gzipped |
Raise error is structures with less than 3 unique sites are used as an input.
Open neutron_factors.csv with gzip library
Raise an error if someone wants neutron scattering past atomic number 96.
I added a few ValueError messages for situations I know won't work. Specifically structures with less than 3 unique sites, and neutron scattering for atomic numbers greater than 96. I'm having trouble thinking of any more edge cases than that. I'm not sure of any specific tests I can add. Also I would like to set up a tutorial for this, as some of it's features are not all that intuitive. What would be the best way to go about setting that up? |
3c23114
to
36e289c
Compare
d725325
to
dca98be
Compare
e3fbc67
to
41e6d99
Compare
Thanks. One thing I would like you to do is to instead of a |
I have written code that takes a list of symmetrized structure objects and generates an f* diagram automatically with the plotly library.
Checklist
ruff
.mypy
.duecredit
@due.dcite
decorators to reference relevant papers by DOI (example)Tip: Install
pre-commit
hooks to auto-check types and linting before every commit: