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

Script extract_renewal_events.py: Fix AtomGroup Info in Output File #187

Merged
merged 1 commit into from
Oct 28, 2023

Conversation

andthum
Copy link
Owner

@andthum andthum commented Oct 28, 2023

Descriptive caption

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.

Proposed changes

In the output file of the script scripts/dynamics/extract_renewal_events.py the information about the reference group was not about the reference but the selection group. This is fixed by this PR.

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.

@github-actions github-actions bot added bugfix Pull requests that fix a bug patch Change that results in a new patch version labels Oct 28, 2023
The info in the text output file about the reference group was actually
about the selection group and not the reference group.  This is fixed by
this commit.
@andthum andthum force-pushed the fix/scripts/dynamics/extract-renewal-events branch from 9b4db64 to 9413403 Compare October 28, 2023 09:18
@andthum andthum merged commit 49a5aed into main Oct 28, 2023
18 checks passed
@andthum andthum deleted the fix/scripts/dynamics/extract-renewal-events branch October 28, 2023 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Pull requests that fix a bug patch Change that results in a new patch version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant