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

Fix some errors introduced by PR #151 #153

Merged
merged 5 commits into from
Mar 10, 2023
Merged

Fix some errors introduced by PR #151 #153

merged 5 commits into from
Mar 10, 2023

Conversation

andthum
Copy link
Owner

@andthum andthum commented Mar 10, 2023

Fix some errors introduced by PR #151

Type of change

  • Change of core package.
  • Change of scripts.
  • Bug fix.
  • New feature.
  • Code refactoring.
  • Dependency update.
  • Documentation update.
  • Maintenance.
  • Other: Description.
  • Non-breaking (backward-compatible) change.
  • Breaking (non-backward-compatible) change.

PR checklist

  • I followed the guidelines in the Developer's guide.
  • New/changed code is properly tested.
  • New/changed code is properly documented.
  • The CI workflow is passing.

@andthum
[MANIFEST.in]: Correct Incomplete Glob Patter
a2e9fde 
Change `prune **/_sphinx_autosummary` to `prune **/_sphinx_autosummary*`
@andthum
[rmsd_trj_trj.py]: Fix
a9c2a9c 
* Remove `nargs=2` from `--weights` argument, because it should only
  require and accept one argument.

* Fix formatting of the header of the output file.
@andthum
[unwrap_trj.py]: Explicitly Copy Arrays
156047d 
Explicitly copy the values from the arrays of the current time step to
the arrays of the previous time step to avoid errors arising from the
fact that Python handles everything as reference if possible.
@andthum
[box.py]: unwrap_frame: Update Examples/Tests
c05ed93 
In the function `mdtools.box.unwrap_frame`, change the examples/test for
the `out` and `out_tmp` arguments from examples with constant boxes to
changing boxes.

Fix a wrong doc sentence.
@andthum
[settings.json]: cSpell: Extend List of Ignored Words
229f6d5
@github-actions github-actions bot added bugfix Pull requests that fix a bug patch Change that results in a new patch version labels Mar 10, 2023
@andthum andthum merged commit 6738d00 into main Mar 10, 2023
@andthum andthum deleted the fix/trajectory-unwrapping-fix branch March 10, 2023 14:22
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
bugfix Pull requests that fix a bug patch Change that results in a new patch version
Projects
None yet
Milestone
No milestone
Development

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