search

Simon-Initiative / course-digest

Tool to produce a summary or digest of OLI course package contents
MIT License
2 stars 0 forks source link

[ENHANCEMENT] convert response mult [MER-2858] #228

Closed andersweinstein closed 10 months ago

andersweinstein commented 10 months ago

This adds support for translating legacy questions using response_mult into new torus response_multi activities. Legacy response_mult allowed for grouping sets of inputs in a multi-input style question and treating those sets as a single part, via response rules that match responses over the set of "target" inputs in that part.

andersweinstein commented 10 months ago

Discovered an issue. It is generating compound rules of the form rule1 && rule2 && rule3 which the torus rule parser apparently does not accept. Torus rule grammar requires rule1 && (rule2 && rule3) and the parentheses may not be omitted. Similarly for disjunctions.

The initial implementation of response_multi in torus also generates rules of this form, so does not work for more than 2 conjuncts or disjuncts.

andersweinstein commented 10 months ago

Parentheses issue now addressed.

rgachuhi commented 10 months ago

Note: this implementation seems to work fine when I use it with a question pool, but I am getting the error "unsupported element [response_mult] detected, exiting" when the response_multi is in a legacy inline activity.