TP · Capturas de cada ejecución en el informe (regla del docente) - #51
Merged
Merged
Conversation
El docente exige una captura de pantalla de cada ejecución: una que solo se nombra en el texto o aparece como listado cuenta como no demostrada, y de ahí salieron las observaciones de la PC2. tp/scripts/capturar_ejecuciones.sh corre el pipeline, pytest, go vet + go test -race, la regresión de Spin, un benchmark reducido y los checks de CI de la PR #50, guarda la salida real en img/ejec-*.txt y la renderiza con capturas_terminal.py. Las salidas de pan de las Secciones 7 y 17 pasan de listados a figuras con su comando, y las Secciones 5, 8, 10, 14 y 17 suman la captura de la ejecución que antes solo nombraban.
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
Reviewer's GuideEl PR agrega un flujo reproducible para ejecutar y capturar cada verificación exigida, y actualiza el informe para presentar esas capturas como evidencia —incluidas las ejecuciones de datos, Go, Spin, benchmark y CI— en lugar de simples listados o menciones. Flow diagram for reproducible execution capturesflowchart TD
A[capturar_ejecuciones.sh] --> B{Selected groups}
B --> C[datos]
B --> D[go]
B --> E[spin]
B --> F[benchmark]
B --> G[ci]
C --> H[Run data pipeline and pytest]
D --> I[Run go vet and go test -race]
E --> J[Run Spin regression and pan outputs]
F --> K[Run reduced benchmark]
G --> L[Run gh pr checks 50]
H --> M[Write ejec-*.txt]
I --> M
J --> M
K --> M
L --> M
M --> N[capturas_terminal.py]
N --> O[Render ejec-*.png]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
El docente pide una captura de pantalla de cada ejecución; una ejecución que solo se nombra o aparece como listado cuenta como no demostrada. De ahí vinieron las observaciones de la PC2.
Qué cambia
tp/scripts/capturar_ejecuciones.sh: corre cada ejecución, guarda su salida real enimg/ejec-*.txt(primera línea: el comando) y la renderiza concapturas_terminal.py. Se puede rehacer por grupos:datos,go,spin,benchmark,ci.pytest(Sec. 5),go vet+go test -racecompleto (Sec. 8), benchmark reducido de demostración (Sec. 10), checks de CI de la PR TP · Cierre del Entregable 3: visor rediseñado, capturas regenerables y PRs del grupo en el informe #50 (Sec. 14),make checkde Spin (Sec. 17).pande las Secciones 7 y 17 pasan de listados de texto a capturas con su comando.Verificación
pytest: 66 pasan.nyc-tlc all: gold con el mismo sha256 (a3ad5205…).go vetygo test -race -count=1 ./...sin-short: ok (≈160 s).make check: 7/7 casos.Notas
gofmtque esa PR arregla. Toca14-github.texen otro párrafo que TP · Cierre del Entregable 3: visor rediseñado, capturas regenerables y PRs del grupo en el informe #50: no debería chocar.Summary by Sourcery
Add reproducible execution captures throughout the report to provide direct evidence for every documented verification run.
New Features:
Enhancements:
Documentation: