Decide whether the catalogue cuts on ShapePipe's external_paired_fraction

Aperta
  • Qualcosa non va?
#444 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@cailmdaley ci sta già lavorando.

Dal 30/10/2026.

  • #446 di @cailmdaley — aperta

Valutazione

Difficoltà
5/5
Tempo stimato
Più di una settimana
Idoneità per principianti
25/100
Tipo di issue
Funzionalità
Chiarezza
Abbastanza chiara
Stato di attività
Ferma
Stack tecnologico
python
Ambito
data

Direzione di ricerca

Read the sp_validation decision record and the catalogue selection code that consumes ShapePipe's tile_make_cat.json; the payload does not name their paths. Start with the existing decision record and the eight-tile diagnostic at /scratch/cdaley/match-guard-diag/report.html. Record whether selection cuts tiles or masks area using external_paired_fraction, and the threshold, or record that no cut is chosen.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Claude Opus on behalf of Cail.

ShapePipe (CosmoStat/shapepipe#981) no longer rejects a tile when DR6 objects fail to pair with SExtractor detections. Those losses are external-only: a DR6 object with no SExtractor detection gets no shape. The tile warns above max(20, 0.02 N + 50) and publishes its counts in tile_make_cat.json beside final_cat: the five match counts, external_paired_fraction and the warning budget, per TILE_ID, including zero-row tiles.

Nothing applies a cut on this downstream, so every object on a sparse edge tile enters the catalogue. 058.220 (71 external-only out of 845) and 207.238 both exceed the warning budget. sp_validation reads no catalogue_match field and records no decision about it.

To decide: whether the catalogue selection cuts tiles, or masks area, on external_paired_fraction, and at what threshold. Leaving them uncut is also a valid decision; if chosen, it gets recorded as one. Either way, the choice goes into sp_validation's decision record. ShapePipe's detection.tile_detection rationale says the cut is undecided and points here.

Evidence: the eight-tile diagnostic is on Nibi at /scratch/cdaley/match-guard-diag/report.html.

Lingua principale
Python
Stelle
2
Fork
5
Merge medio
22h 26m
PR unite (30g)
30

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di CosmoStat/sp_validation

Tutte le issue di CosmoStat/sp_validation

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.