Skip to content

[ WIP ] Add tutorial for membrane RBFE calculation#286

Open
hannahbaumann wants to merge 2 commits intomainfrom
membrane_tutorial
Open

[ WIP ] Add tutorial for membrane RBFE calculation#286
hannahbaumann wants to merge 2 commits intomainfrom
membrane_tutorial

Conversation

@hannahbaumann
Copy link
Contributor

No description provided.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@github-actions
Copy link

github-actions bot commented Feb 6, 2026

Colab 👈 Launch a Colab session on branch membrane_tutorial

@hannahbaumann hannahbaumann changed the title Add tutorial for membrane RBFE calculation [ WIP ] Add tutorial for membrane RBFE calculation Feb 6, 2026
@@ -0,0 +1,1153 @@
{
Copy link
Contributor

@jthorton jthorton Feb 6, 2026

Choose a reason for hiding this comment

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

Line #25.                settings=RelativeHybridTopologyProtocol._adaptive_settings(

Do we not need to add the lipid force field to the settings or is adaptive doing this? Also this is another place we are exposing _adaptive to the world do we need to add a public entry point?


Reply via ReviewNB

@@ -0,0 +1,1153 @@
{
Copy link
Contributor

@jthorton jthorton Feb 6, 2026

Choose a reason for hiding this comment

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

Line #43.    network = openfe.AlchemicalNetwork(transformations)

Woow so many warnings we need to remove a lot of these!


Reply via ReviewNB

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