https://pfspipe.ipmu.jp/jira This file is an XML representation of an issue en-us 8.3.4 803005 13-09-2019 https://pfspipe.ipmu.jp/jira/browse/REDMINE1D-204 1D Redmine <p><em><font color="#505f79"> Created on 2023-05-16 09:03:23 by Vincent Le Brun. % Done: 100</font></em></p> <p>et de deux, il semble y avoir un probleme avec le reader photométrie, dans /net/CESAM/amazed/vlebrun/EL-COSMOS2020/test le fichier errors.csv dit<br/> spec_1288877 0000 Failed to prepare reader : 'photometryBand'<br/> alors que dans le fichier config_fastspec_0.42_phot.json il y a<br/> "photometry_path":"/net/CESAM/amazed/vlebrun/EL-COSMOS2020/COSMOS_2020_master_H24_vizu_2.csv"</p> <p>et que le fichier existe<br/> et que j'ai mis photometry à false dans le parameters_0.42.json</p> REDMINE1D-204

[RM-8063] Pb reader photometrie

Task Normal Done Done Redmine-Jira Migtation Redmine-Jira Migtation Wed, 5 Jul 2023 02:01:25 +0000 Tue, 6 Feb 2024 18:47:01 +0000 Wed, 24 Jan 2024 18:28:08 +0000 0 1 <p>Comment by Mira Sarkis on 2023-05-16 11:52:08:<br/> 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.</p> <p>De plus, pour le reader fastspec, on accède au param.json pour récuperer le param @photometryBand@. <br/> Ce dernier n'est pas défini dans ton fichier de param, d'ou l'erreur!</p> <p>Comment by Pierre-yves Chabaud on 2023-05-16 13:00:33:<br/> Vincent, souhaites-tu utiliser la photométrie dans ce test ?</p> <p>Comment by Vincent Le Brun on 2023-05-16 14:10:18:<br/> Pierre-yves Chabaud wrote in #note-2:<br/> &gt; Vincent, souhaites-tu utiliser la photométrie dans ce test ?</p> <p>pas dans l'immédiat, j'attend encore le catalogue corrigé, mais assez rapidement oui</p> <p>Comment by Vincent Le Brun on 2023-08-29 15:51:26:<br/> je crois que c'est réglé ?</p> <p>Comment by Ali Allaoui on 2023-09-19 08:37:47:<br/> Est ce qu'on jette une exception si photometry_path est défini mais que enablephotometry=false ?</p> <p>Comment by Didier Vibert on 2023-09-19 08:46:38:<br/> Ali Allaoui wrote in #note-7:<br/> &gt; Est ce qu'on jette une exception si photometry_path est défini mais que enablephotometry=false ?</p> <p>tu poses la question pour connaître le statut actuellement, ou pour savoir s'il faut jeter une exception ? </p> <p>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 -&gt; photometry_path)</p> <p>Sinon je dirais que le reader ne lit la photométrie que si enable_photométrie est true, non ?</p> <p>Comment by Fanny Dufresne on 2023-09-20 07:17:39:<br/> Voilà ce que je peux apporter:</p> <ul class="alternate" type="square"> <li>A priori le paramètre photometry_path n'existe pas (plus), il est actuellement nommé "photometryTransmissionDir"</li> <li>Il y a bien un warning de levé si photometryTransmissionDir est défini alors que enable_phtometry est à false</li> <li>Il y a une erreur de levée si photometryTransmissionDir n'est pas défini alors que enable_photometry est à true</li> </ul> <p>Comment by Didier Vibert on 2023-11-22 17:51:06:<br/> à fermer, car sera pris en compte par les refacto de @fdufresne ? </p> <p>Comment by Fanny Dufresne on 2023-11-23 09:21:58:<br/> 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</p> <p>Comment by Didier Vibert on 2023-11-23 12:06:35:<br/> Fanny Dufresne wrote in #note-11:<br/> &gt; 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</p> <p>ok je te l'assigne alors, et le repasse en in progress</p> <p>Comment by Pierre-yves Chabaud on 2024-02-06 14:01:59:<br/> C'est apres le refacto =&gt; closed</p> Development Rank 0|zzsy7r: