feat: Wrap Coach -- rule-based hold/wrap advice (--coach) - #16
Merged
Conversation
This was referenced Jul 4, 2026
Adds wrap_coach.py: a few readable thresholds layered on the SAME recent-window forecast/stall band the rest of Pellet Pilot already uses, so advice never contradicts the printed ETA line. Distinguishes the CURRENT, ongoing stall streak from total cook stall time, escalates past ~90 continuous minutes, and knows the difference between "still stalled, not wrapped" (wrap now) vs "wrapped and still stalled" (bump the grill temp instead). Wired in as poll.py's opt-in --coach / PELLET_PILOT_COACH flag, printed alongside the existing forecast line each tick. Kept separate from the --speak pipeline entirely (not spoken) to avoid touching that already carefully-tuned code path. Live-verified against the actual (completed) pork butt cook's full history, replayed tick-by-tick through the real forecast/stall logic.
ctopherwilliams
force-pushed
the
feat/wrap-coach
branch
from
July 5, 2026 01:47
41392e9 to
4265f8f
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
wrap_coach.py: rule-based mid-cook advice (hold for bark vs. wrap to push through), built on the same recent-window forecast/stall band already driving the ETA -- never contradicts it.poll.py --coach/PELLET_PILOT_COACH=1, printed each tick alongside the existing forecast line. Deliberately kept out of the--speakpipeline to avoid touching that already-tuned code.pelletCLI + YAML cook presets #14 → feat: shareable Cook Report (self-contained HTML) #15 → this one.Test plan