[REDMINE1D-343] [RM-8435] [parameters checker] add a dependance for asym Lya parameters Created: 24/Oct/23  Updated: 06/Feb/24

Status: In Progress
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-10-23 14:56:03 by Didier Vibert. % Done: 50

all the parameters from @LineModelSolve.linemodel.lyafit@ section are necessary only if @LineModelSolve.linemodel.igmfit@ is False (and unused if True)

same for parameters @LineModelSolve.linemodel.lyaforcefit@ and @LineModelSolve.linemodel.lyaforcedisablefit@

note: the section lyafit is read by the instanciation of any fitter (see CAbstractfitter constructor code). We must add a test on the igmfit param to read or not this section (and set the corresponding members to NAN if we don't read).

BTW, in the @CAbstractFitter()@ constructor, these parameters should also be read if the method is @"LineMeasSolve"@. This is currently missing, and the linemeas solver is thus currently running through the Lya asymmetric profile fitting using the hardcoded default values for the parameters ranges and steps...
Note that the default parameter currently used for linemeas is igmfit false (for all parameters, notably the one delivered to PFS)

Thus we need to add the lyafit section in @LineMeasSolve@ with same dependance on igmfit.



 Comments   
Comment by Redmine-Jira Migtation [ 06/Feb/24 ]

Comment by Fanny Dufresne on 2024-02-01 16:18:14:
@dvibert what do you mean by "same for parameters LineModelSolve.linemodel.lyaforcefit and LineModelSolve.linemodel.lyaforcedisablefit" ?

  • lyaforcefit and lyaforcedisable fit are necessary only if igm is false ?
  • lyaforcefit and lyaforcedisable fit are to be added in lineMeasSolve section ?
Comment by Redmine-Jira Migtation [ 06/Feb/24 ]

Comment by Didier Vibert on 2024-02-01 17:46:34:
Fanny Dufresne wrote in #note-3:
> @dvibert what do you mean by "same for parameters LineModelSolve.linemodel.lyaforcefit and LineModelSolve.linemodel.lyaforcedisablefit" ?
>
> - lyaforcefit and lyaforcedisable fit are necessary only if igm is false ?
> - lyaforcefit and lyaforcedisable fit are to be added in lineMeasSolve section ?

yes both !

Comment by Redmine-Jira Migtation [ 06/Feb/24 ]

Comment by Didier Vibert on 2024-02-01 17:58:55:
peut-être implémenter #7688-12 dans la foulée ?

Comment by Redmine-Jira Migtation [ 06/Feb/24 ]

Comment by Fanny Dufresne on 2024-02-02 09:50:00:
En regardant de plus près lyaForceFit et lyaForceDisableFit j'ai l'impression que : (avec => signifie implique)

  • lyaForceDisableFit à true => lyaForceFit et lyaFit section inutiles
  • lyaForceFit à false => lyaFit section inutile

La section lyaFit est finalement utile uniquement si lyaForceDisableFit à false et lyaForceFit à true ?

Comment by Redmine-Jira Migtation [ 06/Feb/24 ]

Comment by Didier Vibert on 2024-02-02 13:44:15:
Fanny Dufresne wrote in #note-7:
> En regardant de plus près lyaForceFit et lyaForceDisableFit j'ai l'impression que : (avec => signifie implique)
> - lyaForceDisableFit à true => lyaForceFit et lyaFit section inutiles
> - lyaForceFit à false => lyaFit section inutile
>
> La section lyaFit est finalement utile uniquement si lyaForceDisableFit à false et lyaForceFit à true ?

cf [AmazedlibParamRefactor_#Renaming]

Generated at Sat Feb 10 19:23:37 JST 2024 using Jira 8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b.