[REDMINE1D-71] [RM-5806] linmodel free/rules: erreur avec plusieurs continus Created: 04/Jun/21  Updated: 05/Jul/23  Resolved: 05/Jul/23

Status: Done
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: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   

Created on 2020-06-03 08:08:23 by Didier Vibert. % Done: 100

lorsqu'on décide de conserver plus d'un continu par ordre de merit pour fitter le linemodel (paramètre @linemodelsolve.linemodel.continuumfit.count@ > 1)

le best chi2 et paramètres du linemodel correspondant retenu à chaque z, dans @CLineModelElementList::fit@, correspondent à ceux du dernier continu testé (donc a priori le moins bon...), au lieu de comparer pour regarder lequel donne le meilleur résultat (cette comparaison est faite dans le cas du tpl-ratio)

cf source:RedshiftLibrary/src/lib/linemodel/elementlist.cpp@master#L2569 où la comparaison est faite pour le cas tpl-ratio, entre les différents templates mais aussi les différents continus



 Comments   
Comment by Redmine-Jira Migtation [ 05/Jul/23 ]

Comment by Didier Vibert on 2020-06-03 08:40:43:
La question est dans le ticket #5875.

Comment by Redmine-Jira Migtation [ 05/Jul/23 ]

Comment by Mira Sarkis on 2022-07-01 16:36:21:
J'ai profité pour faire un minor refactoring de CLineModelFitting::fit et puis de remplacer "tplshape" par "tplratio" .

mr-lib : https://gitlab.lam.fr/CPF/cpf-redshift/-/merge_requests/362
mr-datase-params: https://gitlab.lam.fr/amazed/dataset-parameters/-/merge_requests/30

Comment by Redmine-Jira Migtation [ 05/Jul/23 ]

Comment by Mira Sarkis on 2022-07-04 15:26:37:
Les ITs sont valides, par rapport aux résultats de fix_issue_6961.

Par contre, j'ai identifié des différences avec develop, dans les deux ITs suivants:

  • se8_roman_prism
  • se8_roman_grism,
    qui semble ne pas etre testés/validés dans le cadre 6961!

Note: j'ai réussi à réduire le nombre de ligne de @fit@, en passant de 871 à 671 lignes!

Comment by Redmine-Jira Migtation [ 05/Jul/23 ]

Comment by Didier Vibert on 2022-07-13 14:30:46:
Mira Sarkis wrote in #note-14:
> Les ITs sont valides, par rapport aux résultats de fix_issue_6961.
>
> Par contre, j'ai identifié des différences avec develop, dans les deux ITs suivants:
> * se8_roman_prism
> * se8_roman_grism,
> qui semble ne pas etre testés/validés dans le cadre 6961!
>

normalement les it roman passent maintenant sans changements dans 6961. Tu peux confirmer et verifier que c'est aussi le cas pour cette issue. merci

Comment by Redmine-Jira Migtation [ 05/Jul/23 ]

Comment by Mira Sarkis on 2022-07-13 16:32:06:
Didier Vibert wrote in #note-17:
> Mira Sarkis wrote in #note-14:
> > Les ITs sont valides, par rapport aux résultats de fix_issue_6961.
> >
> > Par contre, j'ai identifié des différences avec develop, dans les deux ITs suivants:
> > * se8_roman_prism
> > * se8_roman_grism,
> > qui semble ne pas etre testés/validés dans le cadre 6961!
> >
>
> normalement les it roman passent maintenant sans changements dans 6961. Tu peux confirmer et verifier que c'est aussi le cas pour cette issue. merci

Oui les IT romans passent maintenant.

Par contre, les variations @lm_continuumComponent_nocontinuum@ présente des différences maintenant!

Comment by Redmine-Jira Migtation [ 05/Jul/23 ]

Comment by Mira Sarkis on 2022-07-18 12:56:56:
lib: merged in @fix_issue_6652@ : @cbd7b9c3@
dataset-params: merged in @fix_issue_6652@ : @031aa0f7@

Note: les ITs présentent des false-positive diff, correspondant au changement du la valeur d'initialisation du tplRatioName qui change de "None" à "undefined".

Comment by Redmine-Jira Migtation [ 05/Jul/23 ]

Comment by Pierre-yves Chabaud on 2022-08-03 08:01:09:
pylibamazed: merged into @develop@ : @4e408666@
dataset-parameters: merged into @develop@ : @8342207@

Generated at Sat Feb 10 15:29:01 JST 2024 using Jira 8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b.