Crucible Path Independence: run the two parent-path scans concurrently (6.4 min serial step)
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 55/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- github-actions, python
- Área
- performance, testing
Línea de trabajo
Start with tests/tools/crucible_path_check.py, especially main() and scan(), and inspect crucible-path-independence.yml to understand the timed step. First measure whether one galaxyscope scan saturates the 4-vCPU runner, then assess the independent scans and required error handling. Done means the full comparison and failure cases still work, with before and after step timings recorded in the PR body.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Measured (2026-10-16 to 2026-10-18, last 8 successful PR runs): crucible-path-independence.yml takes 7.1 min median, 7.5 max. Of that, 6.4 min is the step "Scan the corpus from two parent paths" (tests/tools/crucible_path_check.py --corpus ../language-crucible/data).
Where the time goes: in main(), scan(neutral_dir, ...) and then scan(special_dir, ...) run one after the other. Each is a full python -m gitgalaxy.galaxyscope subprocess over the staged language-crucible copy. The two scans are independent: different staged directories and different output dirs.
Proposed
- Run the two
scan()calls at the same time, for example with twosubprocess.Popens or a 2-workerThreadPoolExecutoraroundscan. Keep the error handling: an unreadable output is still a finding, andneutral_error != special_errorstill counts as a diff. - First measure how many cores one galaxyscope scan already uses on the 4-vCPU runner. If it already saturates them, parallel scans gain little. In that case, look at whether the scan can drop work that never depends on the parent path (e.g. snapshot outputs not compared), and record the finding here.
- Unit test: the check still reports a diff when one side differs, and still fails when one side's output can't be read (fake
scan).
Rule: the same corpus is scanned from both parent paths, and the full comparison runs.
Acceptance: the step's time before and after, in the PR body (target ~3.5–4 min).
- Lenguaje dominante
- Python
- Estrellas
- 57
- Forks
- 11
- Merge medio
- 1 h 5 min
- PR fusionados (30 d)
- 976
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 squid-protocol/gitgalaxy
-
claim:melek priority: high
squid-protocol/gitgalaxy#6534 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
claim:melek priority: high
squid-protocol/gitgalaxy#6501 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
claim:nyx enhancement legacy-modernization scope:core status: blocked testing
squid-protocol/gitgalaxy#6492 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug claim:nyx scope:sweep
squid-protocol/gitgalaxy#6454 · 8 comentarios ·
Los mantenedores suelen responder en 1 día
-
bug claim:melek scope:frontier
squid-protocol/gitgalaxy#6433 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de squid-protocol/gitgalaxy
Issues similares
-
Project submission:Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
slavakurilyak/awesome-ai-agents#845 ·
Los mantenedores suelen responder en 1 día
-
tests: pin that the atomic writers in utils/paths.py stage the temporary file in the target's directoryPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertodifficulty:2 enhancement good first issue
Dificultad 1/5 1-3 horas Aptitud para principiantes 92/100
MakazhanAlpamys/Soup#2178 ·
Los mantenedores suelen responder en 1 día
-
[BUG] dcg.waggr permutation p-values are folded twice, so near-zero scores get the smallest p-valuesAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
scverse/rapids-singlecell#854 ·
Los mantenedores suelen responder en 1 día
-
Slang input: int8_t4_packed and uint8_t4_packed are copied into CrossGLPosiblemente ocupada @ninglinLiu la tomó hoy. Abiertoarea:frontend backend:slang bug direction:input
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 1 día
-
random_elements(unique=True) on an OrderedDict: ZeroDivisionError once only zero weights remain, AssertionError with use_weighting=FalsePosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 13 días