Repository navigation
Route QTI exercises by what each item can express, with one numeric answer reader - #6344
Open
rtibblesbot wants to merge 2 commits into
Open
rtibblesbot wants to merge 2 commits into
rtibblesbot wants to merge 2 commits into
Conversation
`1_000`, `٥`, `5`, `1/2`, `1,000` and `50%` no longer derive a Perseus `numeric-input` with no answers. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
rtibblesbot
force-pushed
the
issue-6294-6996d3
branch
from
October 27, 2026 04:21
8bdf030 to
87653d8
Compare
rtibblesbot
marked this pull request as ready for review
October 27, 2026 04:22
- Perseus archive ships when every item has a Perseus equivalent; QTI package ships for native QTI items or items without one. - When both ship, Perseus items are named by their QTI manifest ids. - Unit pre/post test lists are rewritten to the ids the package ships, leaving out skipped items. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
rtibblesbot
force-pushed
the
issue-6294-6996d3
branch
from
October 27, 2026 07:31
87653d8 to
00c4fc0
Compare
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
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.
Summary
References
Fixes #6294.
Reviewer guidance
Suggested review: Full code review, then manual QA once the reviewer sets Needs QA.
pre_post_testlists andassessment_objectiveskeys may hold QTI ids (K…,item_1) against le-utils' 32-hex schema, and whether a unit version left empty by skipped items should publish without a warning.Number()rejects (1/2,False) from both archives is wanted, and that legacy-only andperseus_question+ native exercises should now ship a Perseus zip as well.Python tests pass on CI; publish was not exercised against a running Kolibri.
pre_post_testlists andassessment_objectiveskeys now hold QTI ids where le-utils' schema says 32-hex; does Kolibri's unit viewer and coach report accept them?AI usage
Used Claude Code to plan, implement and test the change. Verified with the Python test suite and a self-review of the diff.
Deviations from the issue spec
perseus_question) also ship QTI, as acceptance criterion 3 requiresqtiandexercisefiles for themWritten by rtibblesbot, an LLM-based coding agent.
🟡 Waiting for feedback
Last updated: 2026-10-27 07:32 UTC