Galaxy selection: replace the IMAFLAGS_ISO == 0 cut with the MASK_{bit}_{Label} columns
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 65/100
Direzione di ricerca
Inizia da galaxy.py:175, scripts/calibration/extract_info.py:163, scripts/masking.py e dagli elenchi delle colonne in params.py per tracciare ogni uso di IMAFLAGS_ISO. Esamina la tabella dei bit UNIONS di #847 e il collegamento dei cataloghi in #886 e #879, quindi ispeziona i lettori FITS in extract_info.py e compute_leakage_harmony.py. Il lavoro è completato quando i cataloghi v2 non generano più errori relativi a colonne mancanti e la selezione della maschera configurata corrisponde al confronto v1 indicato.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Since CosmoStat/shapepipe#847, ShapePipe generates no masks. Tile detection runs with no flag image, so IMAFLAGS_ISO — which was the rasterized star/galaxy/external mask OR'd over each object's isophote — no longer exists in the shear catalogue. Its replacement is one boolean column per UNIONS mask bit, MASK_{bit}_{Label} (MASK_1_Faint_star_halos … MASK_2048_z2), queried from the sky-fixed healsparse maps at each object's position (True = masked; bit table in #847). The bit/label pairs are the ones sp_validation's ApplyHspMasks already writes into v1 data_ext (4_Stars, …). The wiring lands in CosmoStat/shapepipe#886, and the columns reach final_cat_<campaign>.hdf5 via CosmoStat/shapepipe#879.
sp_validation still cuts on IMAFLAGS_ISO == 0 (galaxy.py classification_galaxy_base, scripts/calibration/extract_info.py, scripts/masking.py, params.py column lists). Against a v2 catalogue that raises a missing-column error.
Proposed: define the galaxy-sample mask as an explicit set of bits and cut on their OR, with the bit list in config rather than code. The set {1, 2, 4, 8, 64, 1024} reproduces the v1 r-band mask on the covered P3 area: faint- and bright-star halos (1, 2), star bodies (4), manual masks (8), the r-band reason bit (64) and MaxiMask (1024). Do not OR every mask column: 16/32/128/256 are per-band (u g i HSC-z) coverage flags, and 2048 is set wherever Pan-STARRS z2 is absent, so a blanket OR masks the whole footprint.
Related: #879 also moves the star catalogue to full_starcat_<campaign>.hdf5 (exposures/<exp> datasets), so the fits.getdata readers in extract_info.py and compute_leakage_harmony.py need an hdf5 path. Both fit naturally with #340.
— Claude (Fable) on behalf of Cail
- Lingua principale
- Python
- Stelle
- 2
- Fork
- 5
- Merge medio
- 14h 50m
- PR unite (30g)
- 27
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di CosmoStat/sp_validation
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
CosmoStat/sp_validation#315 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
CosmoStat/sp_validation#314 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 Mezza giornata Idoneità per principianti 65/100
CosmoStat/sp_validation#272 ·
I maintainer di solito rispondono entro 1 giorno
-
PSF leakage componentsForse già presa @martinkilbinger l’ha presa 3 giorni fa. Aperta
CosmoStat/sp_validation#413 · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
CosmoStat/sp_validation#407 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di CosmoStat/sp_validation
Issue simili
-
Edit tool converts CRLF files to LFForse già presa @RaphaelFakhri l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
huggingface/tau#774 ·
I maintainer di solito rispondono entro 1 giorno
-
show ignores the row limit when a column or table name contains "limit"Forse già presa @RaphaelFakhri l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
I maintainer di solito rispondono entro 2 giorni
-
documentation good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
eclipse-score/inc_someip_gateway#356 ·
I maintainer di solito rispondono entro 2 giorni