Skip to content

Fix measured size followup review regressions#117

Merged
shiny-code-bot merged 1 commit into
mainfrom
fix-size-followup-review-round-two
May 31, 2026
Merged

Fix measured size followup review regressions#117
shiny-code-bot merged 1 commit into
mainfrom
fix-size-followup-review-round-two

Conversation

@shiny-code-bot
Copy link
Copy Markdown
Collaborator

Summary

  • Split measured size-budget alignment so over-target samples can move smaller while under-target samples can spend bitrate on quality.
  • Narrow measured-size follow-up detection to notes with explicit post-sample or smaller-revision context.
  • Add regression coverage for plain size sample notes and under-target quality increases.

Verification

  • uv run --with pytest pytest -q tests/test_tuning_runtime.py -k "plain_size_sample_as_followup or measured_size_followup or quality_increase or quality_tradeoff or under_target_size_budget_quality"
  • uv run --with pytest pytest -q tests/test_tuning_runtime.py
  • bash scripts/pre-commit-check.sh
  • uv run /Users/cbusillo/Developer/codex-skills/jetbrains-inspection/scripts/jb-inspect.py closeout --repo /Users/cbusillo/Developer/mediaforce --scope changed_files

@shiny-code-bot shiny-code-bot merged commit 7582c46 into main May 31, 2026
5 checks passed
@shiny-code-bot shiny-code-bot deleted the fix-size-followup-review-round-two branch May 31, 2026 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants