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.

Sign up for GitHub

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

Jump to bottom

A few fixes to Pylada #63

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

A few fixes to Pylada #63

wants to merge 3 commits into from

Conversation

ftherrien
Copy link
Member

This is just a few tweeks that I have made to pylada in the past year that I forgot to merge.

  1. Fixed the grubber call in neighbors and coordination_shells
  2. Improved primitive, so that it returns an error less often and can handle larger systems in recursive mode
  3. Made it possible to use partial hydrogen e.g. H.75
  4. Fixed the selective dynamics functionality
  5. Fixed the error with nnode that is not an integer

@ftherrien
Fixed the call to gruber in neighbors and coordination_shells
d278c1a 
The second argument to gurber is itermax, not the tolerance. I specifically used the tolerance kwarg instead.
@ftherrien
Added a recursive mode to primitive that can handle much larger struc…
9f3e6ee 
…tures
@ftherrien
Improvements to vasp jobs (re)launching
b871f12 
1. Added correct handling of partial hydrogen atoms (e.g. H.75)
2. Fixed selective dynamics so that the property is retained when launching a vasp job
3. Fixed the infamous float number of nodes error with an integer division
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ftherrien