Skip to content

Merge both new swapping methods together#77

Merged
ajfriedman22 merged 3 commits intoforced-swapfrom
new_swap
Oct 18, 2025
Merged

Merge both new swapping methods together#77
ajfriedman22 merged 3 commits intoforced-swapfrom
new_swap

Conversation

@ajfriedman22
Copy link
Copy Markdown
Collaborator

@ajfriedman22 ajfriedman22 commented Oct 18, 2025

Description

Provide a brief description of the PR's purpose here.

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Unit test and CI update
  • Other

Todos

Notable points that this PR has either accomplished or will accomplish.

  • Add al alternate swapping method where the number of molecules of interest changes rather than changing a portion of a molecule between replicas

Checklist

Starting from PR #62, external PRs must keep the following checklist in the PR.

  • I have read the contributing guidelines.
  • My code follows the project’s coding style and is free from linting errors examined by flake8.
  • My code comes with docstrings that follow the NumPy docstrings standard.
  • I have updated the documentation in the docs directory if my changes affect the project's functionality or usage.
  • I have added new tests for new features or changes to existing features. The code coverage is at least 90%.
  • My code passes all unit tests and CI tests on CircleCI.

@ajfriedman22 ajfriedman22 merged commit 331b558 into forced-swap Oct 18, 2025
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.

1 participant