Closed EdwinB12 closed 2 months ago
Check out this pull request on
See visual diffs & provide feedback on Jupyter Notebooks.
Powered by ReviewNB
This would work better as a method in experiment. The user would run it outside of experiment.process()
Restriction on passed function is it must take in a prompt dictionary and it must return a prompt dictionary
Should support a list/tuple of functions. Don't support arguments. Encourage the user to use the prompt dictionary to parameterise.
This has ended up being a very bare bones application of this and i'm not sure what value it actually adds over just running an evaluation function on the completed responses dictionary saved to disk after called .process()
.
Attention: Patch coverage is 80.00000%
with 2 lines
in your changes missing coverage. Please review.
Project coverage is 52.41%. Comparing base (
cf15ce4
) to head (824734a
). Report is 13 commits behind head on main.
Files with missing lines | Patch % | Lines |
---|---|---|
src/prompto/experiment.py | 80.00% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Very simple application of using an evaluation function in prompto.