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

Update Exscientia Sandpit SOMD config settings #390

Merged
merged 1 commit into from
Feb 13, 2025

Conversation

fjclark
Copy link

@fjclark fjclark commented Feb 12, 2025

Update settings for SOMD in the Exscientia Sandpit to match those in the main version of the code: based on 976ab36 .

This pull request fixes issue OpenBioSim/biosimspace_tutorials#32 .

  • 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

I have run the relevant tests and blackened.

Suggested reviewers:

@lohedges

Thanks

Update settings to match those inn the main version of the code:
see OpenBioSim@976ab36
@lohedges
Copy link
Contributor

Thanks. The failures look to be related to the Sire changes so I'll investigate those locally.

@lohedges
Copy link
Contributor

The CI failure is now fixed in Sire. Thanks again for the update. I hadn't touched the Sandpit when making the optimisation edits since I knew it was used for the tutorials and wasn't sure how it might affect their internal code base, although I don't think they use SOMD. Good to have things aligned now.

@lohedges lohedges merged commit de6629c into OpenBioSim:devel Feb 13, 2025
4 of 5 checks passed
@fjclark
Copy link
Author

fjclark commented Feb 13, 2025

No problem - thank you for fixing.

lohedges added a commit that referenced this pull request Feb 14, 2025
lohedges added a commit that referenced this pull request Feb 14, 2025
lohedges added a commit that referenced this pull request Feb 14, 2025
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.

2 participants