Local recording ends at Finish, before the learning review
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 84/100
- Tipo de issue
- Documentación
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- rust
- Área
- documentation
Línea de trabajo
Lee la sección “Local recordings” en docs/task-mode.md y comprueba su descripción frente al comportamiento indicado aquí: la grabación termina cuando finaliza el intento, antes de que aparezca la revisión de aprendizaje. Añade una frase que explique que la revisión no está incluida y que debe grabarse por separado si se necesita. La tarea estará terminada cuando la documentación describa claramente dónde termina la grabación y qué pueden hacer los estudiantes para grabar la revisión.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Rust
- Estrellas
- 228
- Forks
- 150
- Merge medio
- 1 d 3 h
- PR fusionados (30 d)
- 226
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de sysprog21/codetrial
-
Spot Capacity Round Trips tests accept the two largest rising runsPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
Diagnostic category comment disagrees with its testsPosiblemente ocupada @woohsin la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
sysprog21/codetrial#666 · 1 comentario · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Task editor scroll-sync test fails intermittently in CIPosiblemente ocupada @melonpuff la tomó hace 1 día. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
sysprog21/codetrial#654 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
sysprog21/codetrial#646 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de sysprog21/codetrial
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
asonnino/blockchain-validator#88 ·
Los mantenedores suelen responder en 1 día
-
BUG-MERGE-TO-HDR-RESULT-NAMEAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
storytold/photocraft#14025 ·
Los mantenedores suelen responder en 1 día
-
ExtrudeAbiertoarea:kernel bug needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
bherbruck/solvecraft#16273 · 1 comentario · 1 reacción ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
storytold/vectorcraft#7242 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 1 día