Skip to content

Teilpunktebewertung für ComposeFormula #230

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Jun 27, 2025

Conversation

nimec01
Copy link
Collaborator

@nimec01 nimec01 commented Nov 21, 2024

Vor dem mergen ist noch folgendes zu beachten: #201 (comment)

Closes #201

@jvoigtlaender
Copy link
Member

Lohnt sich denn wirklich die Verwendung von multipleChoice, wenn eigentlich nur zwei Tupel zu vergleichen sind? Ist das nicht ein bisschen Missbrauch der Funktionalität, bzw. wäre bei Bedarf sinnvoller, eine für solche Fälle geeignete Abstraktion (statt multipleChoice) einzuführen?

@nimec01
Copy link
Collaborator Author

nimec01 commented Nov 22, 2024

Ich würde schon sagen, dass es sich lohnt. Sie nimmt das Grading und Feedback ab (das Anzeigen der Lösung könnte hier auch noch eingebaut werden; ich persönlich mag die Reihenfolge der Outputs gerade aber lieber).

Was sollte/würde die "geeignete Abstraktion" denn anders machen?

@jvoigtlaender
Copy link
Member

Mindestens würde sie nicht multipleChoice heißen, aber für Aufgaben verwendet werden, die in keinster Weise Multiple-Choice-Charakter haben. 😄

Will sagen: die Benennung einer Abstraktion ist Teil ihrer Sinnhaftigkeit.

@jvoigtlaender
Copy link
Member

Eine spezifischere Abstraktion zur Verwendung hier könnte auch Teile der Verhaltens von multipleChoice weglassen, die bei einer Aufgabe der Art "zwei verschiedene Formeln sind einzugeben, und jede gibt 50% wenn sie für sich genommen richtig ist" nicht benötigt werden. Siehe auch fmidue/output-blocks#22.

Eine gute rule of thumb könnte sein: Wenn ein Aufgabentyp nicht partialGrade mittels multipleChoiceSyntax implementiert, dann ist auch Implementierung von completeGrade mittels multipleChoice irgendwie off. (Und ja, mir ist bewusst, dass das wohl auch mindestens eine bisherige Verwendung von multipleChoice in Frage stellt, die ich selbst vorgeschlagen hatte.)

@patritzenfeld patritzenfeld merged commit 79baa61 into fmidue:master Jun 27, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Teilpunktebewertung bei ComposeFormula
3 participants