Skip to content

Drop rules take the lowest marks already in - #179

Merged
nessed merged 2 commits into
mainfrom
claude/new-session-j5d3jb
Oct 4, 2026
Merged

nessed merged 2 commits into
mainfrom
claude/new-session-j5d3jb

Conversation

@nessed

@nessed nessed commented Oct 4, 2026

Copy link
Copy Markdown
Owner

With "best 5 of 7" and four papers back (6, 2, 12, 12 of 12), the grade panel struck papers 5 and 6, which haven't happened, and counted the 2/12.

  • countedAssessments (lib/derive.ts): the worst marks already in are dropped first. One mark always stays counting, and any drops left over come off papers still to come, so the group total is still what the outline says.
  • Papers held out only to fill a drop with no mark to land on read "held out", not "dropped", with a line explaining it.
  • Test updated to the new rule, plus the 6/2/12/12 case. MCP_SETUP.md and readmedesign.md updated.

Typecheck, lint and the grading tests pass locally.

🤖 Generated with Claude Code

https://claude.ai/code/session_01MLHMzQJgKySRBgFiKapP9Y


Generated by Claude Code

claude added 2 commits October 4, 2026 20:01
…come back

Until enough pieces in a drop group are marked to say which are lowest, the
ones still to come sit out. Striking them through as 'dropped' read as a
verdict on a paper that has not happened; a line now explains it.

Co-Authored-By: Claude Sonnet 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01MLHMzQJgKySRBgFiKapP9Y
…come

Best 5 of 7 with four back (6, 2, 12, 12 of 12) struck papers 5 and 6, which
have not happened, and counted the 2/12. The worst marks in hand go first now;
one mark always stays, and leftover drops come off papers still to come.

Co-Authored-By: Claude Sonnet 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01MLHMzQJgKySRBgFiKapP9Y
@vercel

vercel Bot commented Oct 4, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
akada Ready Ready Preview Oct 4, 2026 8:04pm UTC

@nessed
nessed merged commit 8308b20 into main Oct 4, 2026
4 checks passed

This branch was successfully deployed

1 active deployment
Preview — 2f80e1b4 Deployed Oct 4, 2026 by vercel[bot]
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