Skip to content

CLI settings and assorted clean up - #329

Merged
jthorton merged 3 commits into
mainfrom
cli_setting_update
Sep 25, 2026
Merged

jthorton merged 3 commits into
mainfrom
cli_setting_update

Conversation

@jthorton

Copy link
Copy Markdown
Contributor

No description provided.

@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

Copy link
Copy Markdown

Colab 👈 Launch a Colab session on branch cli_setting_update

for repeat in {0..2}; do
jobpath="network_setup/transformations/${dirpath}_${repeat}.job"
cmd="openfe quickrun $file -o results/repeat${repeat}/$relpath -d results/repeat${repeat}/$dirpath"
cmd="openfe quickrun $file -o results_${repeat}/$relpath -d results_${repeat}/$dirpath"

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.

This can be reversed but I was trying to standardise how we recommend formatting the results output.

"- [Absolute hydration free energy calculations](https://docs.openfree.energy/en/latest/tutorials/ahfe_tutorial.html)\n",
"- [Molecular Dynamics (MD) simulations](https://docs.openfree.energy/en/latest/tutorials/md_tutorial.html)\n",
"- [Absolute binding free energy calculations](https://docs.openfree.energy/en/latest/tutorials/abfe_tutorial.html)\n",
"- [Relative binding free energy calculations using a Separated Topologies (SepTop) approach](https://docs.openfree.energy/en/latest/tutorials/septop_tutorial.html)\n",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you sort this alphabetically please? I.e. we want the absolutes together

@IAlibay IAlibay left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just the one thing, but I'll approve early

@jthorton
jthorton enabled auto-merge September 25, 2026 10:25
@jthorton
jthorton merged commit 3ae1649 into main Sep 25, 2026
4 checks passed
@jthorton
jthorton deleted the cli_setting_update branch September 25, 2026 10:54
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