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

Release 2023.5.0 #228

Merged
merged 96 commits into from
Dec 16, 2023
Merged

Release 2023.5.0 #228

merged 96 commits into from
Dec 16, 2023

Conversation

lohedges
Copy link
Contributor

This is the PR for the 2023.5.0 release of BioSimSpace.

  • 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 added a test for any new functionality in this pull request: [y]
  • I confirm that I have added documentation (e.g. a new tutorial page or detailed guide) for any new functionality in this pull request: [y]
  • I confirm that I have permission to release this code under the GPL3 license: [y]

fjclark and others added 30 commits August 8, 2023 09:38
This includes specifying the format of the restraints dictionary
and implementing the numerical correction.
Allow correct SOMD config and pert files to be written for multiple
distance restraints. Accompanying tests have also been introduced.
This involved introducing an additional perturbation type
"release_restraint", used exclusively with multiple distance restraints
to remove all distance restraints but the "permanent" distance
restraint.
Ensure that "endmolecule" is written for all SOMD pert files, and
make the tests on the SOMD pert files more detailed (added checks
for start and end atom and molecule lines).
Increase the number of directional clusters until the restraint search
succeeds or hits the maximum.
system

Otherwise this results in 'Error: an illegal memory access was
encountered launching kernel kClearForces'
This addresses code review comments here
#189.
Pass match_water parameter to specialised solvation functions
Do this by using .residues() on the result of the search.
@lohedges lohedges added this to the 2023.5.0 milestone Dec 16, 2023
@lohedges lohedges temporarily deployed to biosimspace-build December 16, 2023 19:40 — with GitHub Actions Inactive
@lohedges lohedges temporarily deployed to biosimspace-build December 16, 2023 19:40 — with GitHub Actions Inactive
@lohedges lohedges temporarily deployed to biosimspace-build December 16, 2023 19:40 — with GitHub Actions Inactive
@lohedges lohedges temporarily deployed to biosimspace-build December 16, 2023 19:40 — with GitHub Actions Inactive
@lohedges
Copy link
Contributor Author

As usual, on release I see an inexplicable test failure for one CI build. This test has never failed before and passed on the previous PR to devel, which is the exact same code other than the updated CHANGELOG and requirements file. Will see if it's reproducible. (The failure is an incorrect set of atom indices for a restraint search, which would imply a different result from a Sire search.)

@lohedges
Copy link
Contributor Author

And on a re-run it passes, which really makes no sense. There is nothing stochastic about this test. I must admit that I'm getting a bit concerned about the number of false positives (or negatives, depending on how you interpret it) from the CI at the moment.

@lohedges lohedges merged commit 7dd4668 into main Dec 16, 2023
5 checks passed
@lohedges lohedges deleted the release_2023.5.0 branch December 16, 2023 21:03
@lohedges
Copy link
Contributor Author

Same failure for Linux Python 3.10 on the build of the release packages. Absolutely baffling.

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.

4 participants