Skip to content

El corte por momento necesita las eras, y la frase que corregi seguia escrita - #1167

Merged
github-actions[bot] merged 1 commit into
developfrom
fix/el-corte-por-momento-y-la-frase-que-sobrevivio
Nov 8, 2026
Merged

github-actions[bot] merged 1 commit into
developfrom
fix/el-corte-por-momento-y-la-frase-que-sobrevivio

Conversation

@frapercan

Copy link
Copy Markdown
Owner

Dos cosas pequenas y una de ellas incomoda.

La frase falsa sobrevivio a su propia correccion

#1166 reescribio el docstring de _query_state con la distribucion medida. Y
dejo viva, pegada al campo donde se emite, exactamente la afirmacion que
corregia:

# Expected to be zero while the axis is release 211's own corpus
# ontology. It is the first thing to read before trusting a bundle.
"discarded_off_axis": st.discards.off_axis,

Son 689.055 filas sobre las 65 ventanas publicadas, el 1,54% de lo crudo.
Corregir un sitio y dejar el otro es peor que no corregir ninguno: el que queda
esta donde lo lee quien use el campo. Lo encontre buscando otra cosa, no
revisando mi propio arreglo, que es la parte que no me gusta.

El bloque explicativo va encima de la funcion y no dentro. Con el dentro,
_emit_state se quedaba en 59 lineas de las 60 del presupuesto, y hoy ya me
paso con update_parent_progress.

Las dos eras, junto al corte que solo se lee contra ellas

El sobremesa necesita, para el informe de cada entrenamiento, el fuera de eje
en t y en t' junto a la antiguedad de cada ventana. El corte por momento
ya lo anadio #1166; la antiguedad no estaba, y no se deduce del id de la
ventana. El evento de queries ya llevaba version_t y version_t_prime;
este no.

Donde queda, para que quede escrito: tabla job_event, evento
transitions.query_state, una fila por ventana, campos window (el id
completo), off_axis_t, off_axis_t_prime, discarded_off_axis, y ahora
version_t y version_t_prime. La era del eje esta en el resultado de la
corrida como cut_version, una sola vez, porque no varia por ventana.

Los tests

Tres, y pinan un contrato con un lector que esta en otro repositorio. Su modo
de fallo es el silencio: un campo que desaparece deja un informe que sigue
dibujandose, con una columna en blanco que nadie mira.

  • el total y sus dos mitades, y que suman;
  • las dos eras;
  • que la ventana se nombra por su id completo de 36 caracteres y no por el
    prefijo de ocho del mensaje, que es el que un lector descuidado cogeria.

Comprobado que el de las eras falla al quitar los dos campos
(KeyError), no solo que pasa con ellos.

Puertas locales: ruff 0, mypy 424 ficheros sin problemas, check_smells 0,
check_no_em_dashes 0, y los 13 tests de query_state en 0.

… escrita

#1166 corrigio el docstring de _query_state y dejo viva la misma afirmacion
falsa donde se emite el campo: un comentario junto a discarded_off_axis decia
que se esperaba cero y que era lo primero que leer antes de confiar en un
bundle. Son 689.055 filas sobre las 65 ventanas publicadas, el 1,54% de lo
crudo. Corregir un sitio y dejar el otro es peor que no corregir ninguno,
porque el que queda esta pegado al campo.

El bloque explicativo va ENCIMA de la funcion, no dentro: con el dentro,
_emit_state se quedaba en 59 lineas de las 60 del presupuesto, y hoy ya me paso
con update_parent_progress.

Y anade las dos eras de la ventana al evento, que es lo que el sobremesa
necesita para el informe de entrenamiento: una caida del 4,54% no se puede leer
sin la antiguedad que sigue, y la antiguedad no se deduce del id de la ventana.
El evento de queries ya las llevaba; este no. Tres tests nuevos fijan el
contrato, porque el lector esta en otro repositorio y su fallo es el silencio:
un campo que desaparece deja un informe que sigue dibujandose con una columna
en blanco. Comprobado que el de las eras falla al quitar los dos campos.
@github-actions
github-actions Bot enabled auto-merge (squash) November 8, 2026 19:59
@github-actions
github-actions Bot merged commit c37115c into develop Nov 8, 2026
17 checks passed
@codecov

codecov Bot commented Nov 8, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 87.99%. Comparing base (d8bc801) to head (d3a85ca).
⚠️ Report is 266 commits behind head on develop.

Files with missing lines Patch % Lines
...otea/core/operations/export_transitions_dataset.py 50.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1167      +/-   ##
===========================================
+ Coverage    87.85%   87.99%   +0.14%     
===========================================
  Files          306      404      +98     
  Lines        23414    31386    +7972     
===========================================
+ Hits         20570    27619    +7049     
- Misses        2844     3767     +923     
Files with missing lines Coverage Δ
...otea/core/operations/export_transitions_dataset.py 56.89% <50.00%> (ø)

... and 166 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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