make_cat: spread-model size check can never fire; SM_DO_CLASSIFICATION silently ignored without its input
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 55/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- Active
- Stack technique
- python
- Domaine
- data-engineering
Piste de recherche
Locate make_cat_runner and save_sm_data in the catalogue-assembly path, then trace the star/galaxy spread-model branch and its three-input path. Verify the size check, w_log call, SPREAD_MODEL output, SM_DO_CLASSIFICATION handling, and missing-threshold error behavior; done means each reported case behaves explicitly and is covered by focused tests.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Found while writing the classification-deferred-downstream contract for the decision record (#875). In the star/galaxy (spread-model) path of catalogue assembly: save_sm_data returns the n_obj argument it was given, so make_cat_runner's SExtractor-vs-spread-model size check compares a value with itself and can never fire — and if it did, it would crash, since it calls w_log(...) as a function; with three inputs (no spread-model catalogue) the runner writes no SPREAD_MODEL column while logging "setting spread model to 99", and SM_DO_CLASSIFICATION = True is silently ignored; a missing SM_STAR_THRESH/SM_GAL_THRESH surfaces as a bare NoOptionError. The path is dormant in the committed workflow (classification off), so nothing measured is affected today.
- Langage dominant
- Python
- Étoiles
- 18
- Forks
- 14
- Merge moyen
- 1 j 4 h
- PR mergées (30 j)
- 18
Préparer son environnement
- Fournit un Dockerfile ou un fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de CosmoStat/shapepipe
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 85/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 65/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de CosmoStat/shapepipe
Issues similaires
-
Broken links found in docsOuvertedocs pydanty:is-working
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
pydantic/pydantic-ai#8863 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
run-llama/llama_index#23278 ·
Les mainteneurs répondent en général sous 2 jours
-
documentation from-review-extraction github-actions priority: low severity:nit
Difficulté 1/5 Moins d'une heure Accessibilité débutants 92/100
LearningCircuit/local-deep-research#6946 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
oracle/langchain-oracle#323 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
tenstorrent/tt-metal#58057 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour