Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
jthorton
left a comment
There was a problem hiding this comment.
Just the two cells to remove and a link to the docs then please merge this looks great!
IAlibay
left a comment
There was a problem hiding this comment.
Can't merge as-is because the environment file is pointing to unreleased versions of openfe and gufe.
|
The "notes" sections could be a little bit more visually striking if we used error and warning boxes. I'll open a PR targetting this one with some of these changes. |
|
@hannahbaumann I added a commit with a note about the CLI support here. Also I see that the CLI is failing on the abfe and septop analysis notebooks, but theyre passing on main. I can look into that further tomorrow. |
|
Thanks @atravitz ! It looks like the tests are failing because we need to update the test data after the changes. Let me know if I should do that or if you want to do it! |
No description provided.