Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

PSFEx validation catalogue: ACCEPTED compares unrelated numberings

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#980 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
74/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
python
Domaine
data

Piste de recherche

Commence dans PSFExInterpolator._write_output_validation dans src/shapepipe/modules/psfex_interp_package/psfex_interp.py, où ACCEPTED est rempli à partir de SOURCE_NUMBER, et confirme la discordance entre la numérotation des lignes d'entraînement et de validation. Suis ensuite la colonne ACCEPTED dans les colonnes facultatives de merge_starcat.py et retire-la des deux chemins de sortie. Terminé signifie que les catalogues de validation n'émettent plus ACCEPTED et que la fusion classique ne la remplit plus avec des zéros, les tests concernés étant ajustés.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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

Langage dominant
Python
Étoiles
18
Forks
14
Merge moyen
4 j 14 h
PR mergées (30 j)
44

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de CosmoStat/shapepipe

Toutes les issues de CosmoStat/shapepipe

Issues similaires

Plus d'issues Python

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.