Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

PSFEx validation catalogue: ACCEPTED compares unrelated numberings

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#980 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 2 Tagen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
74/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
python
Bereich
data

Rechercherichtung

Beginne in PSFExInterpolator._write_output_validation in src/shapepipe/modules/psfex_interp_package/psfex_interp.py, wo ACCEPTED aus SOURCE_NUMBER befüllt wird, und bestätige die Abweichung zwischen der Zeilennummerierung von Training und Validierung. Folge dann der Spalte ACCEPTED in die optionalen Spalten von merge_starcat.py und entferne sie aus beiden Ausgabepfaden. Fertig bedeutet, dass Validierungskataloge ACCEPTED nicht mehr ausgeben und der klassische Merge es nicht mehr mit Nullen auffüllt, wobei betroffene Tests angepasst werden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

The ACCEPTED column of the PSFEx validation catalogues carries no information.
It is not a science bug: rho and tau statistics use the held-out validation stars either way.
But anyone reading ACCEPTED as "PSFEx used this star" is misled.

PSFExInterpolator._write_output_validation (src/shapepipe/modules/psfex_interp_package/psfex_interp.py) sets it like this:

data["ACCEPTED"] = np.ones_like(data["NUMBER"], dtype="int16")
star_used = psfex_cat_dict.pop("SOURCE_NUMBER")
for idx in range(len(data["NUMBER"])):
    if idx + 1 not in star_used:
        data["ACCEPTED"][idx] = 0

SOURCE_NUMBER numbers the rows of the PSFEx training catalogue (.cat).
idx + 1 is the row position in the validation catalogue, which holds the other 20% of stars (setools RAND_SPLIT, RATIO=20).
The two numberings are unrelated, so the test compares a validation row index with training row numbers.

In v2 production, every validation star gets ACCEPTED = 1.
In classic runs, merge_starcat.py zero-fills the column after the pix2world conversion, so v1.4 star catalogues have all zeros.

By construction, a validation star was never offered to PSFEx, so "accepted by PSFEx" is not defined for it.
We propose to drop the column from the validation output and from merge_starcat's optional columns.
If a per-star training flag is wanted, it belongs in the training catalogue, matched on NUMBER.

— Claude (Opus) on behalf of Cail

Vorherrschende Sprache
Python
Sterne
18
Forks
14
Ø Merge
4 T. 16 Std.
Gemergte PRs (30 T.)
35

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus CosmoStat/shapepipe

Alle Issues in CosmoStat/shapepipe

Ähnliche Issues

Weitere Issues zu Python

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.