[REDMINE1D-356] [RM-8476] [parameters checker] wrong warning: Unused parameter LSF GaussianVariablewidthFileName Created: 14/Nov/23  Updated: 22/Nov/23

Status: Open
Project: 1D Redmine
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Task Priority: Normal
Reporter: Redmine-Jira Migtation Assignee: Redmine-Jira Migtation
Resolution: Unresolved Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   

Created on 2023-11-13 12:10:58 by Didier Vibert. % Done: 0

with

<pre>
"LSF":

{ "LSFType": "GaussianVariableWidth", "GaussianVariablewidthFileName": "LSF/roman_grism_lsf.fits" }

,
</pre>

we get the warning "Unused parameter LSF GaussianVariablewidthFileName"



 Comments   
Comment by Redmine-Jira Migtation [ 22/Nov/23 ]

Comment by Didier Vibert on 2023-11-13 12:28:29:
en fait sur le même run (IT roman), j'obtiens d'autres warning "unfair":

continuumRemoval

Warning: Unused parameter continuumRemoval medianKernelWidth
Warning: Unused parameter continuumRemoval medianEvenReflection

alors que continuum.component = "tplfitauto" => il peut basculer en fromspectrum

reliability

Warning: Unused parameter object galaxy reliability_enable
dans le fichier de param il y a "enable_reliability": false, qui est requis, et en revanche avec false la propriété "reliability_model" est inutile (mais pas de warning)

continuumreestimation

Warning: Unused parameter object galaxy LineModelSolve continuumreestimation

continuumreestimation est requis pour linemodelSolve (utilisé systématiquement)

désolé si ces contraintes ont mal été spécifiées initialement, là je regarde ce que fait le code C++ pour conclure.

Comment by Redmine-Jira Migtation [ 22/Nov/23 ]

Comment by Didier Vibert on 2023-11-21 08:33:13:
je viens de regarder les logs des derniers IT full, donc je note dans cette issue, les warnings du checker qui me semble soit manquant, soit injustifiés...
WIP j'éditerai ce commentaire pour finir avec les autres IT

  • se8:
  • continuumRemoval: no warning (only unused warning for continuumRemoval.medianKernelWidth & continuumRemoval.medianEvenReflection), but whole section unused
  • galaxy reliability_enabled: unused warning, but required (idem roman ci-dessus, required for all objects type galaxy/star/qso/...)
  • LinemeasSolve: no warning, but unused.
  • LineModelSolve.continuumreestimation: unused warning, but required (idem roman ci-dessus)
  • LineModelSolve.useloglambdasampling: no warning, but unused (usefull only if continuum.component is tplfit* and continuumfit.fftprocessing true)
  • se8_fromspectrum:
  • continuumRemoval.medianKernelWidth & medianEvenReflection: unused warning, but required (when Templatefitting, tplcombination & LinemodelSolve with continuum.component=tplfitauto or fromspectrum)
  • se8_templatefitting_fft:
  • LSF: no warning (for whole section, only sourcesize): whole section unused (LSF section required only for LinemodelSolve)
  • se8_variable_lsf:
  • LSF.GaussianVariablewidthFileName: unused warning, but required
Generated at Sat Feb 10 15:31:53 JST 2024 using Jira 8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b.