Skip to content
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

584 sql fix restliche TODOs in catquiz.php #612

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from

Conversation

ralferlebach
Copy link
Contributor

Liefert beim Ausführen auf dem LTI-Server eine extrem lange Abfrage mit knapp 60.000 Einträgen (bei 55.672 Fragen im System)
grafik

Wenn die Anfrage mit dem Default-Kontext (nur einer!) ausgeführt wird, werden bei Fragen, die nie in einem CAT-Quiz gespielt wurden (weil sie nicht Teil einer CAT-Skala sind) attempts angezeigt (Kopfdaten gekürzt):
grafik

und bei Fragen, die tatsächlich Teil einer CAT-Skala sind, werden diese angezeigt, oder auch nicht:
grafik

oder aber es werden keine attempts zurückgegeben:
grafik

und manchmal, wenn kein CAT-Kontext angegeben ist, auch einfach beides (obwohl die Frage in einer CAT-Skala zugeordnet ist):
grafik

Mir ist im Allgemeinen auch nach Einblick in den Quelltext nicht vollends klar, was diese Abfrage grundsätzlich bezwecken soll. Und im Besonderen frage ich mich, wieso die CAT-Scalen zwischen zwei Bindestriche concencatet werden, statt eines GROUP-CONCAT mit Separator ','?

Die SQL-Anfrage liefert 60.000 Antworten mit z.t. skurrilen und unplausiblen Werten!
@ralferlebach ralferlebach changed the base branch from main to develop August 12, 2024 07:24
@ralferlebach
Copy link
Contributor Author

ralferlebach commented Aug 12, 2024

Das neue SQL in PR #612 verbessert ein wenig die Laufzeit
grafik

aber das Grundproblem, dass inhärent konfligierende Zielstellungen in einem Statement zu lösen versucht wird, bleibt:

  1. Ermitteln der aktuellen Fragenversion (und keiner älteren)
  2. Ermitteln von Attempt-Statistiken (möglichst für alle Frageversionen)
  3. Abruf sämticher (auch schwergewichtiger) Fragen-Informationen, einschließlich Fragentext oder zugeordneter Kategorie

Immerhin ließen sich einige der Merkwürdigkeiten bereinigen:

Es werden nun alle Fragen - abgesehen von abhängigen Subfragen wie beim qtype cloze - berücksichtigt, nicht nur eine eratisch erscheinende Auswahl:
grafik

Wenn Item-Informationen vorliegen, wird eine Anzahl an Versuchen ausgegeben - falls (noch) keine Versuche vorliegen, dann ist diese (numerisch) Null.
grafik

CAT-Skalen werden nach PR #614 durchgehend zugeordnet.
grafik

@ralferlebach
Copy link
Contributor Author

Disclaimer:

Die Doppelungen in den Einträgen waren noch aus dem letzten Entwicklungsstand des Statements, wo noch nach CATSkalen gruppiert wurde. Ist in der aktuellen Fassung nicht mehr der Fall.

grafik

@ralferlebach ralferlebach changed the title 584 sql fix addcatscalequestions - still WIP 584 sql fix restliche TODOs in catquiz.php Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant