Repository navigation
Publish mixed exercises QTI only when Perseus can't read a legacy input answer - #6273
Conversation
|
Outside this PR's scope — tracked under #5998:
@rtibblesbot's comments are generated by an LLM, and should be evaluated accordingly How was this generated?
|
87f94c1 to
0caa761
Compare
11d98de to
03f92b9
Compare
|
Outside this PR's scope — tracked in:
Written by rtibblesbot, an LLM-based coding agent. |
03f92b9 to
e7186db
Compare
rtibbles
left a comment
There was a problem hiding this comment.
Please rebase this on latest unstable, it seems awfully similar to a recently merged PR.
|
@rtibbles Rebased on unstable. This was stacked on #6262, which has merged, so the diff is now three commits:
The description's "Stacked on #6262" line and its second bullet describe #6262's work, not this diff. |
241f64d to
1497555
Compare
- JSON-number answers (4, 0, 4.5) were published as answers: [] - Answers marked "correct": false were published as correct Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…as answerless Publishing strips unpackageable images before rendering, so an answer of only such an image left a numeric-input with no answers in the archive. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… input A legacy input answer Perseus can't read, such as "Sphere" or "1e400", left the Perseus numeric-input with no answers; such nodes now skip the Perseus archive. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
1497555 to
9020774
Compare
|
Rebased on
|
|
Quick check on rtibblesbot's work on this PR: react 👍 helpful · 😕 mixed · 👎 not helpful. Reply with anything specific. |
Summary
Moves Perseus input-answer parsing onto QTI's
format_number, replacing #6255'sisfinitefilter and #6259's_perseus_number; Perseus and QTI now read an input answer the same way.Sphere,+50%,1e400); otherwise they keep their Perseus archive.4,0), reads+3/1E3as numbers, and no longer publishes"correct": falseanswers as correct.References
Closes #6208. Closes #6266. Refs #6268. Refs #6232.
Reviewer guidance
4fixture QTI only._process_answersoverride moved into the base generator (utils/assessment/base.py), so both formats keep a JSON0.AI usage
Used Claude Code to reproduce the bug with a scratch publish test, write the fix and its tests, and draft this description. Verified with the export channel, exercise creation and QTI test suites, and pre-commit.
🤖 Generated with Claude Code
Written by rtibblesbot, an LLM-based coding agent.
🟡 Waiting for feedback
Last updated: 2026-10-16 22:35 UTC