Skip to content

fix(3823): correct partial/multi-slice tally#4028

Merged
AceHack merged 1 commit into
mainfrom
codex/pr3823-rebased-vera-20260516
May 17, 2026
Merged

fix(3823): correct partial/multi-slice tally#4028
AceHack merged 1 commit into
mainfrom
codex/pr3823-rebased-vera-20260516

Conversation

@AceHack
Copy link
Copy Markdown
Member

@AceHack AceHack commented May 17, 2026

Summary

Replacement for closed/unmerged PR #3823.

Corrects the partial/multi-slice tally from 5 (26%) to 6 (32%) after Codex and Copilot both flagged math drift.

Context

The previous PR #3823 was closed without merge on 2026-05-16. This branch preserves the corrected commit on a fresh Codex-owned ref: codex/pr3823-rebased-vera-20260516.

Verification

  • Branch pushed and verified on GitHub at ff84f13f855e301b831cd7862340775506c8b237
  • Replacement worktree clean before PR creation
  • Root checkout treated as contested; no root writes performed

Copilot AI review requested due to automatic review settings May 17, 2026 02:42
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Single-line correction to a historical tick file fixing the partial/multi-slice tally from 5 (26%) to 6 (32%) and adding the class breakdown (4+1+1) to reconcile math drift flagged by prior reviewers.

Changes:

  • Update count and percentage for partial/multi-slice row
  • Append class-decomposition annotation (Class 2 + 3 + 4 = 4 + 1 + 1)

@AceHack
Copy link
Copy Markdown
Member Author

AceHack commented May 17, 2026

Vera CI note for replacement PR #4028:

  • This PR diff only contains docs/hygiene-history/ticks/2026/05/16/0758Z.md.
  • Current required failures are inherited/out-of-scope for this replacement branch:
    • lint (tick-shard relative-paths): new broken links are in docs/hygiene-history/ticks/2026/05/17/0221Z.md.
    • lint (tsc tools): failures are in tools/inventory/amazon-orders-extract.ts after the Amazon extractor landed.

Holding the PR in draft until those mainline hygiene blockers are fixed or the branch is rebased onto a fixed main. I am not patching those paths in #4028 to keep the PR scoped to the PR-3823 tally correction.

AceHack added a commit that referenced this pull request May 17, 2026
* claim: task-mainline-lint-unblock-4028-20260517 - inherited CI lint unblock

Co-Authored-By: Codex <noreply@openai.com>

* fix(ci): unblock inherited tick links and amazon extractor tsc

Co-Authored-By: Codex <noreply@openai.com>

* release: task-mainline-lint-unblock-4028-20260517 - ready for inherited CI unblock PR

Co-Authored-By: Codex <noreply@openai.com>

---------

Co-authored-by: Codex <noreply@openai.com>
@AceHack AceHack force-pushed the codex/pr3823-rebased-vera-20260516 branch from ff84f13 to 8250aa2 Compare May 17, 2026 07:11
@AceHack AceHack marked this pull request as ready for review May 17, 2026 07:15
Copilot AI review requested due to automatic review settings May 17, 2026 07:15
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@AceHack AceHack merged commit 80d5fce into main May 17, 2026
49 checks passed
@AceHack AceHack deleted the codex/pr3823-rebased-vera-20260516 branch May 17, 2026 07:22
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