enhance: add possibility to check for single exact solution on numeri…

…cal questions (#4333)
uzh-bf · Oct 24, 2024 · b5f558c · b5f558c
1 parent 21bb92f
commit b5f558c
Show file tree

Hide file tree

Showing 5 changed files with 28 additions and 9 deletions.
diff --git a/apps/frontend-manage/src/components/questions/QuestionEditModal.tsx b/apps/frontend-manage/src/components/questions/QuestionEditModal.tsx
@@ -183,14 +183,16 @@ function createValidationSchema(t: ReturnType<typeof useTranslations>) {
               Yup.object().shape({
                 min: Yup.number()
                   .nullable()
+                  // TODO: include this validation step again, once exact solutions can be handled through a separate input
                   // we can only handle one case to avoid cyclic dependencies
                   .when('max', {
                     is: (max?: number | null) => typeof max !== 'undefined',
                     then: (schema) =>
-                      schema.lessThan(
-                        Yup.ref('max'),
-                        t('manage.formErrors.NRMinLessThanMaxSol')
-                      ),
+                      schema.test({
+                        message: t('manage.formErrors.NRMinLessThanMaxSol'),
+                        test: (value, context) =>
+                          value ? value <= context.parent.max : true,
+                      }),
                   }),
                 max: Yup.number().nullable(),
               })

diff --git a/apps/frontend-manage/src/components/sessions/evaluation/QuestionEvaluation.tsx b/apps/frontend-manage/src/components/sessions/evaluation/QuestionEvaluation.tsx
@@ -190,20 +190,35 @@ function QuestionEvaluation({
                 </UserNotification>
               )}
               {showSolution &&
-                currentInstance.questionData.options.solutionRanges && (
+                currentInstance.questionData.options.solutionRanges &&
+                (currentInstance.questionData.options.solutionRanges.length ===
+                  1 &&
+                currentInstance.questionData.options.solutionRanges[0].min &&
+                currentInstance.questionData.options.solutionRanges[0].max &&
+                currentInstance.questionData.options.solutionRanges[0].min >
+                  currentInstance.questionData.options.solutionRanges[0].max -
+                    2 * Number.EPSILON ? (
+                  <div className="mt-4 font-bold">
+                    {t('manage.evaluation.correctExactSolution', {
+                      value:
+                        currentInstance.questionData.options.solutionRanges[0]
+                          .min,
+                    })}
+                  </div>
+                ) : (
                   <div>
                     <div className="mt-4 font-bold">
                       {t('manage.evaluation.correctSolutionRanges')}:
                     </div>
                     {currentInstance.questionData.options.solutionRanges.map(
                       (range, innerIndex) => (
                         <div key={innerIndex}>
-                          [{range?.min || '-∞'},{range?.max || '+∞'}]
+                          [{range.min ?? '-∞'},{range.max ?? '+∞'}]
                         </div>
                       )
                     )}
                   </div>
-                )}
+                ))}
             </div>
           )}
           {currentInstance.questionData.type === 'FREE_TEXT' &&

diff --git a/packages/grading/src/index.ts b/packages/grading/src/index.ts
@@ -93,8 +93,8 @@ export function gradeQuestionNumerical({
   if (definedSolutionRanges.length === 0) return null
 
   const withinRanges = definedSolutionRanges.map(({ min, max }) => {
-    if (min && response < min) return false
-    if (max && response > max) return false
+    if (min && response < min - Number.EPSILON) return false
+    if (max && response > max + Number.EPSILON) return false
     return true
   })
 

diff --git a/packages/i18n/messages/de.ts b/packages/i18n/messages/de.ts
@@ -1429,6 +1429,7 @@ Da die KlickerUZH-App noch nicht im iOS-App-Store verfügbar ist, folgen Sie die
       fontSize: 'Schriftgrösse',
       validSolutionRange: 'Erlaubter Antwortbereich',
       correctSolutionRanges: 'Korrekte Lösungsbereiche',
+      correctExactSolution: 'Korrekte Lösung: {value}',
       statistics: 'Statistik',
       keywordsSolution: 'Schlüsselwörter Lösung',
       noChartsAvailable: 'There exists no chart for this question type yet',

diff --git a/packages/i18n/messages/en.ts b/packages/i18n/messages/en.ts
@@ -1416,6 +1416,7 @@ Since the KlickerUZH app is not yet available on the iOS App Store, follow these
       fontSize: 'Font size',
       validSolutionRange: 'Valid solution range',
       correctSolutionRanges: 'Correct solution ranges',
+      correctExactSolution: 'Correct solution: {value}',
       statistics: 'Statistics',
       keywordsSolution: 'Solution keywords',
       noChartsAvailable: 'There exists no chart for this question type yet',