Comment by Ali Allaoui on 2023-06-23 08:20:53:
j'ai comparé les it el_cosmos et el_cosmos_galaxy_linemodel_enablephotometry et il y a bien des différences (tag f2b7ca80, donc un peu après la 0.42, mais rien qui aurait pu réparé la photométrie
amzdiff voit bien des différences entre /net/CESAM/amazed/vlebrun/SelfCal/output_0.42_phot_debug /net/CESAM/amazed/vlebrun/SelfCal/output_0.42_wide_long_lambda_PSF12 mais très faibles, les plus importantes sont sur LeastSquare (diff relative> 0.1), donc je suppose que ça vient des données photo

Comment by Didier Vibert on 2023-06-23 08:50:55:
ne serait-ce pas un problème d'unité ? les données photo doivent être des flux en Erg/s/cm2/Hz (en supposant que le spectre est bien en Erg/s/cm2/Angstrom, ie sans facteur 1e-16 qui traine)

Comment by Vincent Le Brun on 2023-06-23 09:02:14:
j'avais fait la conversion avant de les ingérer et j'ai vérifié sur quelques cas que les valeurs sont OK. Et si elles étaient complètement fausse, sur le run avec un poids de photométrie à 1, les résultats devraient être bizarres, pas identiques à la spectro seule ...

Comment by Didier Vibert on 2023-06-23 09:03:30:
afin de débugger on peut faire tourner cet exemple dans la branche #7767 en attente de merge.
Cette branche rajoute pour chaque candidats les flux photométriques (dans les unités du spectre avec la conversion 1/Angstrom -> 1/Hz) correspondant au fit (ie le template avec son amplitude intégré dans les bandes photo).

En attendant que vizu sache afficher ça.

Comment by Pierre-yves Chabaud on 2023-06-23 09:05:20:
On est pret pour faire le merge de #7767

Comment by Vincent Le Brun on 2023-06-23 14:23:19:
bon il semblerait que le code soit bien protégé contre les valeurs de photométrie aberrante (genre un facteur 10^10 vu que c'est ce qu'il y a entre les flux /A et /Hz), vu que quand je remet les bonnes unités il se passe des choses (pas forcément rassurantes mais c'est une autre histoire).

Comment by Didier Vibert on 2023-06-26 12:24:41:
Vincent Le Brun wrote in #note-6:
> bon il semblerait que le code soit bien protégé contre les valeurs de photométrie aberrante (genre un facteur 10^10 vu que c'est ce qu'il y a entre les flux /A et /Hz), vu que quand je remet les bonnes unités il se passe des choses (pas forcément rassurantes mais c'est une autre histoire).

ok, pas sur que ce soit une protection explicite ! mais je penche plutôt pour des additions de quantités de l'ordre 10^-20 (valeur au carré) qui ne change donc rien au résultat !