Skip to content

Conversation

@atravitz
Copy link
Contributor

@atravitz atravitz commented Oct 24, 2025

looks like this one hasn't been touched since v1.0.0!

downstream of #244

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@atravitz atravitz requested a review from IAlibay October 24, 2025 00:23
@atravitz atravitz force-pushed the update-t4lysozyme branch 2 times, most recently from 04ead18 to 06bf7d6 Compare October 24, 2025 02:07
Copy link
Member

@IAlibay IAlibay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm approving this, but I would like us to just rermove the duplicate examples under openmm_rbfe, they aren't needed for anything & and you found out, we're forgetting about them.

@atravitz atravitz merged commit dedc0c8 into main Oct 24, 2025
2 of 4 checks passed
@atravitz atravitz deleted the update-t4lysozyme branch October 24, 2025 14:12
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.

3 participants