Local recording ends at Finish, before the learning review
Maintainer antworten meist innerhalb von 1 Tag
Bewertung
- Schwierigkeit
- 1/5
- Geschätzter Aufwand
- Unter einer Stunde
- Anfängerfreundlichkeit
- 84/100
- Issue-Typ
- Dokumentation
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- rust
- Bereich
- documentation
Rechercherichtung
Lies den Abschnitt „Local recordings“ in docs/task-mode.md und prüfe seine Beschreibung anhand des hier gemeldeten Verhaltens: Die Aufzeichnung endet, wenn der Versuch abgeschlossen ist, bevor die Lernbesprechung erscheint. Füge einen Satz hinzu, der erklärt, dass die Besprechung nicht enthalten ist und bei Bedarf separat aufgenommen werden muss. Erledigt, wenn die Dokumentation klar beschreibt, wo die Aufzeichnung endet und was Lernende tun können, um die Besprechung aufzunehmen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Observed behavior
The local task recorder stops as soon as the attempt is finished, so the downloaded WebM ends right after the "Finish and freeze this revision" dialog and never shows the learning review. docs/task-mode.md says that completion finalizes the recording, so the behavior matches the documentation.
Why it matters
The course assignment asks for a recording "from the rules page to the learning review after Finish". The built-in recorder cannot capture that last part, and a new capture cannot be started during the attempt.
Steps to reproduce
- Start the server with
cargo run --release -- web --task-modeand openhttp://localhost:3000in Chrome. - Accept the rules, allow the microphone and camera, and calibrate.
- On the preparation page, start the local recording, choose this tab and share tab audio.
- Press Start, work on the task, press Finish and confirm.
- Open the downloaded WebM. Its last frame shows the workspace just after the dialog closes; the learning review that the page shows next is not in the file.
Cause
In web/task.js, attemptOver() calls stopMonitoring(), which calls localRecording.finish() and releases the camera and microphone. The result arrives later and renderReview() shows it, after the recorder has already saved the file.
Plan
The change I can send is a documentation one: a sentence in the "Local recordings" section of docs/task-mode.md stating that the recording ends when the attempt is finished and therefore does not include the learning review, so a learner who needs the review on video knows to capture it separately.
Recording through the review would be a larger change: for an attempt that ends with Finish, keep the tab capture running after monitoring stops, let the recorder drop its microphone input without reporting a lost microphone, and finalize in renderReview() once the review is in the page, with browser tests for both paths. I cannot commit to that before the assignment deadline, so I am leaving it as a direction rather than a promise.
Environment
codetrial 0.1.0 (5570fe9), built from source; server in WSL Ubuntu on Windows 11; Chrome 157.0.8103.39; default task rubik, language C.
- Vorherrschende Sprache
- Rust
- Sterne
- 228
- Forks
- 150
- Ø Merge
- 1 T. 3 Std.
- Gemergte PRs (30 T.)
- 226
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus sysprog21/codetrial
-
Spot Capacity Round Trips tests accept the two largest rising runsEvtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
Maintainer antworten meist innerhalb von 1 Tag
-
Diagnostic category comment disagrees with its testsEvtl. vergeben @woohsin hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 66/100
sysprog21/codetrial#666 · 1 Kommentar · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
Task editor scroll-sync test fails intermittently in CIEvtl. vergeben @melonpuff hat das vor 1 Tag übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
sysprog21/codetrial#654 · 2 Kommentare · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 88/100
sysprog21/codetrial#646 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in sysprog21/codetrial
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
horizon-rust/horizon#16176 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
asonnino/blockchain-validator#88 ·
Maintainer antworten meist innerhalb von 1 Tag
-
BUG-MERGE-TO-HDR-RESULT-NAMEEvtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offenbug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
storytold/photocraft#14025 ·
Maintainer antworten meist innerhalb von 1 Tag
-
enhancement good first issue
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 85/100
-
ExtrudeOffenarea:kernel bug needs-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
bherbruck/solvecraft#16273 · 1 Kommentar · 1 Reaktion ·