Skip to content

Fix residue naming#2042

Open
hannahbaumann wants to merge 8 commits into
mainfrom
fix_residue_naming
Open

Fix residue naming#2042
hannahbaumann wants to merge 8 commits into
mainfrom
fix_residue_naming

Conversation

@hannahbaumann

Copy link
Copy Markdown
Contributor

Checklist

  • All new code is appropriately documented (user-facing code must have complete docstrings).
  • Added a news entry, or the changes are not user-facing.
  • Ran pre-commit: you can run pre-commit locally or comment on this PR with pre-commit.ci autofix.

Manual Tests: these are slow so don't need to be run every commit, only before merging and when relevant changes are made (generally at reviewer-discretion).

Developers certificate of origin

@hannahbaumann

Copy link
Copy Markdown
Contributor Author

pre-commit.ci autofix

@codecov

codecov Bot commented Jun 29, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.92473% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 90.61%. Comparing base (541e1ca) to head (91fde04).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...s/protocols/openmm_rfe/test_hybrid_top_protocol.py 98.21% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2042      +/-   ##
==========================================
- Coverage   94.95%   90.61%   -4.35%     
==========================================
  Files         217      218       +1     
  Lines       20768    20864      +96     
==========================================
- Hits        19720    18905     -815     
- Misses       1048     1959     +911     
Flag Coverage Δ
fast-tests 90.61% <98.92%> (?)
slow-tests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@hannahbaumann

Copy link
Copy Markdown
Contributor Author

pre-commit.ci autofix

@hannahbaumann hannahbaumann changed the title [WIP] Fix residue naming Fix residue naming Jun 29, 2026
@hannahbaumann hannahbaumann linked an issue Jun 29, 2026 that may be closed by this pull request
@github-actions

Copy link
Copy Markdown

No API break detected ✅

View workflow run

Griffe output
$ griffe check "openfe" -s src --no-inspection --no-color --verbose -a origin/main

$ griffe check "openfecli" -s src --no-inspection --no-color --verbose -a origin/main

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.

Change ligand and cofactor residue names in the topology

1 participant