Skip to content

La declaracion a 92866b6: la evaluacion sale del hilo principal - #354

Merged
frapercan merged 1 commit into
mainfrom
decl/92866b6-la-evaluacion-y-el-tsv
Oct 11, 2026
Merged

frapercan merged 1 commit into
mainfrom
decl/92866b6-la-evaluacion-y-el-tsv

Conversation

@frapercan

Copy link
Copy Markdown
Owner

Decimoseptimo salto, de 1eb70c3 a 92866b6. Dos PR de correccion de PROTEA.

#1004, la descarga del TSV mandaba un trozo HTTP por fila. 2.235.489 mensajes
ASGI para 43,6 MB en known-terms.tsv. Medido en proceso: 172,83 s contra 0,21 s
con bloques de 64 KB, 823x, salida identica byte a byte.

#1005, la evaluacion no podia correr fuera del hilo principal, y decia que si.
run_cafa_evaluation nunca ha producido una metrica en esta campana: el unico
evaluation_result de la base es el de la prueba de humo de hoy, con los tres
compartimentos vacios y el trabajo en SUCCEEDED.

  • Causa: _invoke_cafaeval_signal_safe llama a signal.signal, que CPython
    rechaza fuera del hilo principal, y _job_pump.py:61 corre cada operacion en un
    hilo por trabajo.
  • Puerta: un ajuste que fallaba se registraba como WARNING y aportaba {}.

Para el nodo el salto es checkout y reinicio. Ni migraciones ni modelos entre los
dos shas, la base sigue en f2a8c41d9e37, y pyproject.toml y poetry.lock sin
tocar, asi que protea-sources se queda en fc43ab94. Cinco ficheros, dos de
ellos de tests.

…TSV deja de trocear por filas

Decimoseptimo salto. Dos PR de correccion de PROTEA, #1004 y #1005.

#1005 es la que importa: run_cafa_evaluation nunca ha producido una metrica
en esta campana. signal.signal fuera del hilo principal mataba los tres
ajustes, y la puerta los degradaba a WARNING, asi que el trabajo decia
SUCCEEDED con tres celdas vacias. Para el nodo el salto es checkout y
reinicio; ni migraciones ni hermanos.
@frapercan
frapercan merged commit 6f23a77 into main Oct 11, 2026
7 checks passed
@frapercan
frapercan deleted the decl/92866b6-la-evaluacion-y-el-tsv branch October 11, 2026 18:16
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