* add more checks
* make precommit manual
* apply formatting to pyproject.toml
* add TODO
* remove unneeded, add a few more
* add ruff, but turn everything off
* add openfe known first party
* format highest-level files
* first half of openfe protocols
* second half of openfe protocols
* openfe protocols formatting, with alyssa's fmt skips
* add ruff formatter to precommit
* fmt: off all vendored _rfe_utils code
* addressing review comments
* format openfe/utils
* format openfe/setup
* first batch of cli formatting
* second batch of cli formatting
* formatting the rest of openfecli commands
* format openfecli/parameters
* format openfe/storage
* run precommit
* Update openfecli/commands/gather.py
Co-authored-by: Irfan Alibay <IAlibay@users.noreply.github.com>
* update example notebooks branch for v1.7.0 docs changes (#1615)
* bump example notebooks branch
* add ipykernel to env
* roll back to fixing septop branch
* i dont think we want ipykernel
* bump to tmp_fix_docs branch
* point to branch revert-237-v1.7_cookbooks
* point to latest example notebooks release
* remove colab button, point to updated example notebooks, reorg landing page (#1618)
* remove colab button from example notebooks in docs
* point to example notebooks 2025.10.2
* replace 'try' with CLI
---------
Co-authored-by: Irfan Alibay <IAlibay@users.noreply.github.com>
* escape string correctly
* Fix test to check if string is printed
This test used to use pytest.warns to check if we printed a string to
the terminal. This isn't the right way to test if things get printed to
terminal since it only checks warnings raised in python-land. The test
passed since the regex used to match warnings ended up matching warnings
raised since it was a negated character class which matches a single
character that is not any of the characters listed inside.
I've seen a few cases in the wild where CI fails because a file failed
to download. I think we should try bumping to 5 since with each retry it
increases the wait time and I think that ~3s might be too fast so I want
to try 5s.
Co-authored-by: Irfan Alibay <IAlibay@users.noreply.github.com>
Co-authored-by: Alyssa Travitz <31974495+atravitz@users.noreply.github.com>
* add pandas construction
* removing csv writer
* replicating original behavior
* add rich table
* split into function
* make names clearer
* adding adjustments from rebase
* put it in a box
* adding tsv flag
* update tests to use tsv flag
* add smoke test for pretty print
* adding file creation test
* adding news item
---------
Co-authored-by: Irfan Alibay <IAlibay@users.noreply.github.com>
Co-authored-by: Mike Henry <11765982+mikemhenry@users.noreply.github.com>
* print message to stderr when no valid results files are found
* fix check
* move check into main function to keep single responsibility
* fix cmet test bug
* adding test
* reorg for clarity
* moving try/except into function for clarity
* reorg data parsing for clarity
* remove unused function
* add try/except to pass through info about failed edges
* updating tests for new expected failed simulation checking behavior
* pass through for raw and ddg
* pass through failures for dgs
* remove unused import
* adding a missing check
* docstrings
* cleaning up logic
* cleaning up logic
* docstrings and formatting
* fixing conditionals bug
* more docstrings
* make 'name' a tuple for consistency
* make functions private
* updating output statement for clarity (hopefully)
* remove unused dg raw expected output
* adding comment description
* updating tests
* cleaning up test to not use tmp dir
* only use non-None legs for deciding whether to do rbfe/rhfe
* fix typo
* pass empty list instead of None through to ddg
* add cmet data
* add assert success tests - still need output validation
* light reordering for clarity
* adding pytest regressions
* don't include stderr in file regression
* making output more legible
* switch from runtime error to exit(1)
* add missing complex legs test
* adding regressoin tsvs to manifest
* reordering functions for clarity
* updating tests
* adding test for disconnected network
* updating warning message, not throwing error for now
* updating tests for missing runs
* add disconnected network check
* switch from NaN to Error
* removing unused regression files
* fixing typo
* adding test coverage
* fix typo
* adding news entry
* updates to cli from_json
* replacing dump instances with to_json
* Remove unnecessary imports of JSON_HANDLER
* addressing review
---------
Co-authored-by: Mike Henry <11765982+mikemhenry@users.noreply.github.com>
Co-authored-by: David L. Dotson <dotsdl@gmail.com>
* adding output for failed edges
* adding print statements, tests don't pass yet because of extra output
* remove straggler
* Update openfecli/commands/gather.py
Co-authored-by: Josh Horton <Josh.Horton@newcastle.ac.uk>
* comment out exception check temporarily
* adding exception check back (test data was wrong)
* adding tests in progress
* trying different mocking
* adding unit tests
* cleaning up
* updating zenodo doi to use correct parallel rbfe dataset
---------
Co-authored-by: Josh Horton <Josh.Horton@newcastle.ac.uk>
* switch to assert_click_success to make debugging easier
* fixing tests to look for logger.INFO instead of warnings (due to gufe change)
* updating expected output due to gufe changes
* updating expected transformations
* remove easy prefix, passing 2fe7741b9448
* define _settings_cls, tests pass against g3c6d5f9
* update string match for partial charge logging
* test out pooch
* adding zenodo dois
* remove unused import
* delete rbfe_result data from local storage
* move into a fixture factory
* updating missing legs tests
* Revert "delete rbfe_result data from local storage"
This reverts commit 1211200947e38ad17df50ebc3e67eb801e0bf0a4.
* updating failed edges tests
* updating test names to rbfe
* adding zenodo fetchable
* splitting out docs func
* move data out of fixture
* Revert "adding zenodo fetchable"
This reverts commit 5c72321e0e60d09339aee726c06bcda1190a6beb.
* add check for data cache
* cleaning up straggling changes
* remove data that's now on zenodo
* adding correctly compressed results data
* be able to manually trigger package tests
* run package tests on each pr, no reason not to
* we should be installing from conda forge
* skip some openfecli tests if we don't have nagl installed
* plz lets use conda-forge
* add type annotations
* use assert_click_success
* rename set_vals to leg_types for clarity
* formatting stdout whitespace for clarity
* formatting output to handle multiple legs in a clearer way
* docstring
* adding news item
* updating test match strings
* simplify legs error message
* updating tests for new error handling
