add results handling for new worker-based execution

[atravitz]

resolves #2119

LLM / AI generated code disclosure

LLMs or other AI-powered tools (beyond simple IDE use cases) were used in this contribution: no

Checklist

• All new code is appropriately documented (user-facing code must have complete docstrings).
• Added a news entry, or the changes are not user-facing. (this will happen before merge into main)
• Ran pre-commit: you can run pre-commit locally or comment on this PR with pre-commit.ci autofix.
• Filled in the AI generated code disclosure.

Manual Tests: these are slow so don't need to be run every commit, only before merging and when relevant changes are made (generally at reviewer-discretion).

• GPU integration tests
• example notebook testing
• packaging tests: run this for any large feature PRs or PRs that add test data.

Developers certificate of origin

• I certify that this contribution is covered by the MIT License here and the Developer Certificate of Origin at https://developercertificate.org/.

[github-actions]

🚨 API breaking changes detected! 🚨

View workflow run

Griffe output

$ griffe check "openfe" -s src --no-inspection --no-color --verbose -a origin/main
src/openfe/tests/storage/test_resultclient.py:0: <module>:
Public object was removed

src/openfe/tests/storage/test_metadatastore.py:0: <module>:
Public object was removed

src/openfe/tests/storage/test_resultserver.py:0: <module>:
Public object was removed

src/openfe/storage/metadatastore.py:0: <module>:
Public object was removed

src/openfe/storage/resultserver.py:0: <module>:
Public object was removed

src/openfe/storage/resultclient.py:0: <module>:
Public object was removed


$ griffe check "openfecli" -s src --no-inspection --no-color --verbose -a origin/main
src/openfecli/commands/plan_rbfe_network.py:121: plan_rbfe_network(warehouse):
Parameter was added as required

[codecov]

Codecov Report

❌ Patch coverage is 86.95652% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.61%. Comparing base (c11378c) to head (391ae10).

Files with missing lines	Patch %	Lines
src/openfe/storage/warehouse.py	78.78%	14 Missing ⚠️
src/openfe/orchestration/__init__.py	66.66%	1 Missing ⚠️

Additional details and impacted files

@@                     Coverage Diff                      @@
##           epic/execution_improvements    #2114   +/-   ##
============================================================
  Coverage                        90.60%   90.61%           
============================================================
  Files                              219      219           
  Lines                            21608    21692   +84     
============================================================
+ Hits                             19579    19657   +78     
- Misses                            2029     2035    +6     

Flag	Coverage Δ
fast-tests	90.61% <86.95%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[ethanholz]

I like the implementations I see here. The results model makes sense.