Repository navigation
Conversation
6679bdf to
218da73
Compare
218da73 to
81a3c48
Compare
|
Assigning reviewers: R: @jrmccluskey for label python. Note: If you would like to opt out of this review, comment Available commands:
The PR bot will only process comments in the main thread (not review comments). |
|
Run PythonDocs PreCommit |
jrmccluskey
left a comment
There was a problem hiding this comment.
This is a neat framework! Just a few documentation nits and a question around the potential for malformed responses
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #40220 +/- ##
============================================
+ Coverage 56.16% 56.36% +0.19%
Complexity 2288 2288
============================================
Files 1124 1127 +3
Lines 178336 178650 +314
Branches 1489 1488 -1
============================================
+ Hits 100170 100691 +521
+ Misses 75645 75435 -210
- Partials 2521 2524 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Adds typed Choice, Boolean, and Score questions to Beam Python.
EvaluateDecisionsapplies aDecisionModeladapter to each input element, so pipelines can change adapters while keeping their questions and downstream policy.The transform reuses the adapter context across elements and bundles on each worker DoFn instance. Results retain the original input, timestamps, and windows, with model/provider identifiers and request latency. The transform records request metrics and closes the context during best-effort teardown.
The optional Jev adapter maps Boolean questions to Noul and validates responses. The fraud example's optional JSONL error output catches malformed-response validation failures. Setup errors and exhausted transport failures still fail that pipeline. Calls and retries block the processing thread, and the transform does not coordinate a global request-rate limit.
The README covers installation, local demos, and cloud options.
Local mocked validation and the routing and fraud demos passed. Python 3.12 ML CI passed with the optional SDK installed for the adapter tests. Provider calls were not repeated for the response-validation changes, and the routing example's BigQuery sink has not been tested against the live service.
Run the ML unit suite with:
addresses #123orfixes #456).CHANGES.mdwith noteworthy changes.See CONTRIBUTING.md for contribution guidelines.