Skip to content
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

Update get-primitive for modern versions of spglib #7

Merged
merged 11 commits into from
May 29, 2024
Merged

Conversation

ajjackson
Copy link
Owner

@ajjackson ajjackson commented May 29, 2024

Closes #3

  • spglib no longer supports Atoms as input
  • while we are here, modernise the input arguments a bit
  • make the printed output cleaner
  • remove some vasp5-related logic that is no longer needed.

TODO

  • Unit test

I'm not running black on everything (yet...) but black-macchiato can
by a nice tool for formatting while editing. I prefer to stick to
79-char lines at this stage, so putthing this in pyprojct.toml makes
life easy.
- create an intermediate function that receives/returns ASE
  Atoms. This should be useful for Python scripting, as well as breaking
  up the main program here.

- Simplify "verbose" printing logic

- Iteration over atoms.cell doesn't require "array" to be addressed
@ajjackson ajjackson merged commit d23103b into develop May 29, 2024
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

spglib ~=2.0 breaks the package
1 participant