Comment by Didier Vibert on 2023-07-07 09:23:12:
after discussions this morning:
instead of having two keys (an integer ID and the composed string str_Id), we keep inside the C++ class a unique integer key.
When we add a new line in the catalog (in C++ called by python api) we check the unicity of the str_Id, and then forget about it (we keep the list only for checking unicity in add). Then all direct access to a specific line are done via the integer id and the linecatalog will be a map instead of a vector of lines

Comment by Didier Vibert on 2023-09-14 14:03:38:
MR: https://gitlab.lam.fr/CPF/cpf-redshift/-/merge_requests/540

Comment by Didier Vibert on 2023-09-19 13:39:17:
gros problème d'augmentation du temps de calcul du à l'accès indirect des raies dans des std::map au lieu de std::vector

a priori revenir à un accès direct au moins dans les CElement.

Comment by Didier Vibert on 2023-09-29 13:33:47:
le fix de l'utilisation des id des raies semble corriger un pb linemeas (cf #8248) dû à un mélange des raies dans le hybridfitter concernant la détermination des raies qui se chevauchent et donc du polynôme commun pour fitter le continu...

sur le test d'intégration sp8_linemeas_lbfgsb
avant:

après:

Comment by Vincent Le Brun on 2023-09-29 14:25:15:
parfait... une fin de vendredi comme on les aime...

Comment by Didier Vibert on 2023-09-29 14:43:45:
Vincent Le Brun wrote in #note-17:
> parfait... une fin de vendredi comme on les aime...

peut être que ça corrige aussi le fit foireux du QSO que tu as vu. C'était en linemodel "free" ou "ratio" ?

Comment by Vincent Le Brun on 2023-09-29 14:44:55:
ratio. Dès que c'est en merged dans un vent je ferai un test. Mais ça n'aurait pas du arriver plus souvent ?

Comment by Didier Vibert on 2023-09-29 14:51:50:
Vincent Le Brun wrote in #note-19:
> ratio. Dès que c'est en merged dans un vent je ferai un test. Mais ça n'aurait pas du arriver plus souvent ?

si il y avait un pb avec linemeas autour de OII depuis longtemps...

Comment by Vincent Le Brun on 2023-09-29 15:25:26:
mais mon probleme avec le solver QSO n'a rien à voir avec OII, à moins qu'il y ait eu un mélange d'autres raies dans les linecatalog QSO. Tu as moyen de vérifier ça sur le spectre DESI?

Comment by Didier Vibert on 2023-10-02 14:16:50:
IT full

Comment by Pierre-yves Chabaud on 2023-10-03 08:56:05:
Merged into @develop@ (ef67328c)