[REDMINE1D-10] [RM-6499] le prior 'H-alpha' n'est pas optimal Created: 04/Jun/21  Updated: 13/Jun/23  Resolved: 13/Jun/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 2021-05-17 13:53:31 by Vincent Le Brun. % Done: 100

ce prior prend environ deux fois plus de temps que le prior strong, il ne serait pas codé de façon 'optimale' d'après Didier...



 Comments   
Comment by yuki.moritani [ 13/Jun/23 ]

Comment by Mira Sarkis on 2022-02-01 15:50:35:
Je confirme que le temps de traitement de Haprior est double que le traitement des stronglinepriors:
Sur 12 spectres pfs, on passe de 33 secondes (stronglinepriors activé) à 82 secondes (Haprior activé) par spectre, sur cpu_4216.
Note que c'est de 34 sec avec le prior EuclidHa.

  • PFS7: no Prior : 32.85275636
  • PFS7: Haprior : 81.93655360000001
  • PFS7: stronglinePrior : 33.051047319999995
  • PFS7: EuclidHaprior : 34.071445319999995

Pour les prior Halpha, il y a une étape en plus dans ::BuildChisquareArray qui sert à chercher la raie la plus forte en amplitude et puis de verifier si c'est Halpha. Il s'agit d'une boucle sur la grille fine en z (17820 valeurs pour euclid) et puis une autre boucle sur les raies (53 raies par e.g) et ceci pour tous les tplratios (130 au total).

Le fait de basculer cette recherche au moment du fitting des tplratio, nottament pour la firstpass, réduit le temps de traitement surtout que la grille de la firstpass (1620 z + 5*200 pour intervalles secondpass) est beaucoup plus petite que celle de la grille fine.
Sur le meme subset de pfs7, cette solution réduit le temps de traitement à 36 sec avec haprior activé.

Note that no variation with Haprior exists in IT!

Comment by yuki.moritani [ 13/Jun/23 ]

Comment by Mira Sarkis on 2022-02-03 13:13:58:
Added a new IT with haprior activated, for one se8 spectra whose best redshift = 1.379, ie., in the visible range of Halpha.

Dataset-param merge-request:
https://gitlab.lam.fr/amazed/dataset-parameters/-/merge_requests/14

Lib merge-request:
https://gitlab.lam.fr/CPF/cpf-redshift/-/merge_requests/265/commits

Comment by yuki.moritani [ 13/Jun/23 ]

Comment by Pierre-yves Chabaud on 2022-02-22 08:25:12:
Merged into @develop@ : @3799de1@

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