Comment by Mira Sarkis on 2023-05-16 11:52:08:
Le bug c'est que le client essaye de charger la photométrie sans regarder le param @enablephotometry@, et ceci dès que @photometry_path@ est présent dans le fichier de config.

De plus, pour le reader fastspec, on accède au param.json pour récuperer le param @photometryBand@.
Ce dernier n'est pas défini dans ton fichier de param, d'ou l'erreur!

Comment by Pierre-yves Chabaud on 2023-05-16 13:00:33:
Vincent, souhaites-tu utiliser la photométrie dans ce test ?

Comment by Vincent Le Brun on 2023-05-16 14:10:18:
Pierre-yves Chabaud wrote in #note-2:
> Vincent, souhaites-tu utiliser la photométrie dans ce test ?

pas dans l'immédiat, j'attend encore le catalogue corrigé, mais assez rapidement oui

Comment by Vincent Le Brun on 2023-08-29 15:51:26:
je crois que c'est réglé ?

Comment by Ali Allaoui on 2023-09-19 08:37:47:
Est ce qu'on jette une exception si photometry_path est défini mais que enablephotometry=false ?

Comment by Didier Vibert on 2023-09-19 08:46:38:
Ali Allaoui wrote in #note-7:
> Est ce qu'on jette une exception si photometry_path est défini mais que enablephotometry=false ?

tu poses la question pour connaître le statut actuellement, ou pour savoir s'il faut jeter une exception ?

Pour 2 je pense que ça fait partie du checker/refactor codé par @fdufresne et de la politique adoptée concernant tous les paramètres "inutiles" c'est à dire présent dans le fichier mais non-utilisé. Elle a programmé pour l'instant un warning, qu'on pourra transformer en erreur, si on veut. Il faut vérifier qu'elle a bien codé cette dépendance (enablephotometry -> photometry_path)

Sinon je dirais que le reader ne lit la photométrie que si enable_photométrie est true, non ?

Comment by Fanny Dufresne on 2023-09-20 07:17:39:
Voilà ce que je peux apporter:

• A priori le paramètre photometry_path n'existe pas (plus), il est actuellement nommé "photometryTransmissionDir"
• Il y a bien un warning de levé si photometryTransmissionDir est défini alors que enable_phtometry est à false
• Il y a une erreur de levée si photometryTransmissionDir n'est pas défini alors que enable_photometry est à true

Comment by Didier Vibert on 2023-11-22 17:51:06:
à fermer, car sera pris en compte par les refacto de @fdufresne ?

Comment by Fanny Dufresne on 2023-11-23 09:21:58:
Je serais plutôt d'avis de le garder pour le moment et vérifier après le refacto que le comportement est bien OK

Comment by Didier Vibert on 2023-11-23 12:06:35:
Fanny Dufresne wrote in #note-11:
> Je serais plutôt d'avis de le garder pour le moment et vérifier après le refacto que le comportement est bien OK

ok je te l'assigne alors, et le repasse en in progress

Comment by Pierre-yves Chabaud on 2024-02-06 14:01:59:
C'est apres le refacto => closed