Complete HookSession input and gradient hook coverage - #129
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #129 +/- ##
=======================================
Coverage 98.30% 98.31%
=======================================
Files 52 52
Lines 3947 3968 +21
=======================================
+ Hits 3880 3901 +21
Misses 67 67 ☔ View full report in Codecov by Harness. |
There was a problem hiding this comment.
All reported issues were addressed across 3 files
You’re at about 93% of the monthly reviewed-line limit. You may want to disable incremental reviews to conserve quota. Reviews will continue until that limit is exceeded. If you need help avoiding interruptions, please contact contact@cubic.dev.
Reply with feedback, questions, or to request a fix.
Re-trigger cubic
Summary
HookSession.captureandHookSession.replacetarget forward inputs, including(args, kwargs)input hooksHookProgramValidation
just checksjust tests— 621 passed, 1 skippeduv run pytest tests/test_demo_notebooks.py -q -m integration— 3 passed, 1 deselectedjust docs— passes with the existing AutoAPI cyclic-import warningCloses #115