[REDMINE1D-329] [RM-8390] [parameters checker] add a new constraint on linemeas velocityfit Created: 06/Oct/23 Updated: 05/Dec/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-10-05 13:09:56 by Didier Vibert. % Done: 0 when the linemeas method is piped after the linemodelsolve, the parameters "emvelocityfitmin" & "emvelocityfitmax" of the linemeas section (if velocityfit is true) should define a range which at least include the range [emvelocityfitmin", "emvelocityfitmax] of the linemodelSolve section. see #8365-10 idem for absvelocityfitmin" & absvelocityfitmax" we should decide if this is an error or just a warning. |
| Comments |
| Comment by Redmine-Jira Migtation [ 05/Dec/23 ] |
|
Comment by Didier Vibert on 2023-10-19 14:02:44: On part donc sur un warning si la plage de vitesse linemeas ne couvre pas celle de redshift-estimation. La différence avec les warnings sur les "unused" parameters, c'est que dans ce cas il faut modifier les paramètres de façon à ce que la plage linemeas couvre celle de z-estim. On peut soit tout faire au niveau du parameters checker en python, ou alors juste le check/warning dans le checker python et modif du param effectif dans le code C++ lors de son utilisation. Il y a un warning aussi à ajouter (soit dans l'API, soit dans le code C++) lorsque la vitesse initiale (qui vient de estim-z ou du linemeas catalog en entrée) n'est pas dans la plage. Et idem, dans ce cas on modifie la plage pour inclure la vitesse initiale. |