Skip to content

Conversation

@lohedges
Copy link
Contributor

This PR closes #496 by updating Sire makeCompatibleWith calls to use the new API, i.e. make_compatible_with. This issue wasn't caught since the calls use try/except, since they can fail for valid reasons.

  • 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]

@akalpokas

@lohedges lohedges added bug Something isn't working cresset Related to work with Cresset labels Jan 23, 2026
@lohedges lohedges temporarily deployed to biosimspace-build January 23, 2026 15:18 — with GitHub Actions Inactive
@lohedges lohedges temporarily deployed to biosimspace-build January 23, 2026 15:18 — with GitHub Actions Inactive
@lohedges lohedges temporarily deployed to biosimspace-build January 23, 2026 15:18 — with GitHub Actions Inactive
@lohedges lohedges temporarily deployed to biosimspace-build January 23, 2026 15:18 — with GitHub Actions Inactive
@akalpokas
Copy link
Contributor

Thanks @lohedges , I've tested this PR and can confirm it fixes the problem.

@lohedges lohedges merged commit b530cb1 into devel Jan 23, 2026
4 of 5 checks passed
@lohedges lohedges deleted the fix_496 branch January 23, 2026 15:51
lohedges added a commit that referenced this pull request Jan 23, 2026
lohedges added a commit that referenced this pull request Jan 23, 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 cresset Related to work with Cresset

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Protein dynamics fails when built using recent version of BioSimSpace

3 participants