Skip to content

update septop settings - #334

Merged
atravitz merged 2 commits into
mainfrom
maint/update_septop_settings
Oct 6, 2026
Merged

atravitz merged 2 commits into
mainfrom
maint/update_septop_settings

Conversation

@atravitz

@atravitz atravitz commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

updating breaking changes from OpenFreeEnergy/openfe#2128

@atravitz
atravitz requested a review from hannahbaumann October 5, 2026 17:26
@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown

Colab 👈 Launch a Colab session on branch maint/update_septop_settings

Comment thread environment.yaml

- pip:
- git+https://github.com/OpenFreeEnergy/openfe@main # use conda-forge v1.13.0 after release
- git+https://github.com/OpenFreeEnergy/openfe@issue-2236 # use conda-forge v1.13.0 after release

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

we can point this back to main once 2236 is merged, but I want to see tests pass.

@hannahbaumann hannahbaumann left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @atravitz lgtm!

@atravitz
atravitz marked this pull request as ready for review October 6, 2026 14:27
@atravitz
atravitz merged commit 7213779 into main Oct 6, 2026
3 checks passed
@atravitz
atravitz deleted the maint/update_septop_settings branch October 6, 2026 14:28
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