feat(plotnine): implement waffle-basic - #9953
Closed
github-actions[bot] wants to merge 2 commits into
Closed
Conversation
Regen from quality 79. Addressed:
- data values now sum to exactly 100 (39/28/20/13) instead of 90,
closing the ~10% proportional gap (SC-02/DQ-03)
- canvas corrected to canonical square 2400x2400 (figure_size=(6,6),
dpi=400) — previous figsize=(16,9)/dpi=300 was historical drift,
not on the Step 0 contract
- added subtitle with a realistic $5,000 monthly-budget scenario and
per-category dollar amounts in the legend for data storytelling
(DE-03)
- added an explanatory caption ("each square = 1% of budget") using
plotnine's labs(caption=...) for reader guidance
- switched to theme_void() base + bold/italic typographic hierarchy
(title/subtitle/caption) for more intentional typography (DE-01/02)
- blanked legend key background so swatches sit flush on the page
surface instead of a default grey box
- kept the perfect theme adaptation, Imprint palette order, and
contiguous reading-order block layout from the previous version
Contributor
Author
🔧 AI Review Produced No Score — Auto-RetryingThe Claude Code Action ran but didn't write |
Contributor
Author
❌ AI Review Failed (auto-retry exhausted)The AI review action completed but did not produce valid output files. Auto-retry already tried once. What happened:
Manual rerun: |
github-actions
Bot
added
ai-review-failed
Contributor
Author
❌ AI Review Failed (auto-retry exhausted)The AI review action completed but did not produce valid output files. Auto-retry already tried once. What happened:
Manual rerun: |
MarkusNeusinger
added a commit
that referenced
this pull request
Aug 5, 2026
…review (#10179) ## The bug `impl-review.yml` used quality score `0` as its sentinel for *"the AI review produced no output"*. But `0` is also a score the review prompt **mandates**: the Stage 1 auto-reject gates in `prompts/workflow-prompts/ai-quality-review.md` require exactly `Score = 0, verdict = REJECTED` for **AR-08** (clipped element) and **AR-09** (mandatory title not visible). Prompt and workflow therefore contradicted each other, and the pipeline was guaranteed to dead-end on exactly the plots it is designed to reject hardest. ## The deadlock chain 1. Plot renders without a visible title → **AR-09** → reviewer returns `Score: 0/100`, `Verdict: REJECTED` — **this is correct behaviour** 2. `Extract quality score` normalised it: `::warning::Invalid quality score '0', defaulting to 0` 3. `Validate review output` fired on `score == '0'` → `::error::AI Review did not produce valid output files` → `ai-review-failed` → `exit 1` 4. `exit 1` skipped **`Add verdict label and take action`** — which the file itself documents as *"the pipeline's only hand-off point: every downstream workflow (merge, repair) starts from a call made right here"* 5. So **`impl-repair` was never dispatched** and the missing title was never fixed 6. `impl-review-retry.yml` rescued once → the re-review scored `0` again (deterministically — the plot was unchanged) → `ai-review-failed` re-applied 7. `ai-review-failed` + `ai-review-rescued` matches **no** watchdog case (`watchdog-stuck-jobs.yml:116` only emits a `::warning::` and defers to a human) → **PR stranded permanently** ## Evidence Six open PRs sit in exactly that state, each with an AR-09 verdict already in hand: | PR | Library / spec | Reported score | Verdict | Gate | |---|---|---|---|---| | #10152 | seaborn windrose-basic | 0 | REJECTED | AR-09 | | #10130 | muix streamgraph-basic | 0 | REJECTED | AR-09 | | #10009 | matplotlib wireframe-3d-basic | 0 | REJECTED | AR-09 | | #10003 | d3 ternary-basic | 0 | REJECTED | AR-09 | | #9968 | plotnine treemap-basic | 0 | REJECTED | AR-09 | | #9776 | matplotlib polar-basic | 0 | REJECTED | AR-09 | This was never an infrastructure failure. In run [31035492709](https://github.com/MarkusNeusinger/anyplot/actions/runs/31035492709) the Claude action reported `"subtype": "success"`, `"is_error": false`, 15 turns, `permission_denials_count: 0` — and posted a complete review ending in `### Score: 0/100` / `### Verdict: REJECTED`. 94 of the last 100 `impl-review` runs are green; the 6 failures are these gate-tripping plots. ## The fix Output presence becomes its own signal, decoupled from the score value: - `Extract quality score` now emits **`has_output`** alongside `score`. `0` is accepted as a valid score; only a non-numeric or out-of-range value marks output as missing. - The six gates that keyed off `score != '0'` / `score == '0'` now key off `has_output`. - A score of `0` therefore flows into the normal `ai-rejected` → `impl-repair` path (threshold floor is 50, so `0 < 50` → rejected → repair dispatched), and only genuinely absent output raises `ai-review-failed`. **Second, latent bug fixed in the same step:** the comment fallback read `.comments[-1].body`, but on a retry the workflow's own *"auto-retrying"* notice is posted **after** the review — so the fallback searched the notice and found no score. It now selects the last `claude[bot]` comment. **Deliberately not changed:** `watchdog-stuck-jobs.yml`. With the root cause fixed, "review produced no output twice in a row" (PRs #9953/#9952/#9951, which have no `claude[bot]` comment at all) is a genuine failure that *should* escalate to a human rather than loop forever. ## Verification GitHub Actions changes have no verification loop in this repo, so the step's shell body was tested directly: a harness extracts the `Extract quality score` `run:` block **verbatim from the YAML** and exercises it with a stubbed `gh`. ``` --- the regression that caused the deadlock --- PASS file score 0 (AR-09 auto-reject) score=0 has_output=true PASS comment fallback, score 0 score=0 has_output=true --- normal operation must be unchanged --- PASS file score 87 / 100 / 1 / trailing-newline 73 score=... has_output=true PASS comment fallback, score 87 score=87 has_output=true --- genuine 'no output' must still be detected --- PASS no file, no review comment score=0 has_output=false PASS no file, comment without a score line score=0 has_output=false PASS file with non-numeric garbage score=0 has_output=false PASS file with out-of-range score score=0 has_output=false PASS empty file score=0 has_output=false ALL CASES PASS ``` YAML validity re-checked after the edit (`yaml.safe_load`, 20 steps parsed). Residual risk: the `if:` expression rewrites and the `REPOSITORY` env addition are only observable on a real pipeline run. Recovery path for the six stranded PRs after merge: re-dispatch `impl-review.yml -f pr_number=<n>`, which will now score them 0, label `ai-rejected`, and hand them to `impl-repair` to fix the titles. 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
Owner
|
Closing as superseded.
This PR stalled on the score- 🤖 Closed during PR-queue cleanup. |
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.
Implementation:
waffle-basic- python/plotnineImplements the python/plotnine version of
waffle-basic.File:
plots/waffle-basic/implementations/python/plotnine.pyParent Issue: #998
🤖 impl-generate workflow