Skip to content

Fix GLYCAM force field merging#506

Merged
lohedges merged 4 commits intodevelfrom
fix_glycam
Mar 18, 2026
Merged

Fix GLYCAM force field merging#506
lohedges merged 4 commits intodevelfrom
fix_glycam

Conversation

@lohedges
Copy link
Contributor

This PR builds on OpenBioSim/sire/pull/408 by updating the intrascale merge to use the new C++ mergeIntrascale routine. The PR also adds some optimisations to the ring breaking / growing connectivity checks that caused performance issues for large protein merges.

  • I confirm that I have merged the latest version of devel into this branch before issuing this pull request (e.g. by running git pull origin devel): [y]
  • I confirm that I have permission to release this code under the GPL3 license: [y]

@lohedges lohedges added bug Something isn't working enhancement New feature or request labels Mar 18, 2026
@lohedges lohedges temporarily deployed to biosimspace-build March 18, 2026 09:01 — with GitHub Actions Inactive
@lohedges lohedges temporarily deployed to biosimspace-build March 18, 2026 09:01 — with GitHub Actions Inactive
@lohedges lohedges temporarily deployed to biosimspace-build March 18, 2026 09:01 — with GitHub Actions Inactive
@lohedges lohedges temporarily deployed to biosimspace-build March 18, 2026 09:01 — with GitHub Actions Inactive
@lohedges lohedges temporarily deployed to biosimspace-build March 18, 2026 09:01 — with GitHub Actions Inactive
@lohedges lohedges merged commit 5db39f1 into devel Mar 18, 2026
5 checks passed
@lohedges lohedges deleted the fix_glycam branch March 18, 2026 09:37
lohedges added a commit that referenced this pull request Mar 18, 2026
lohedges added a commit that referenced this pull request Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant