Modernised track material parametrizer with dd4hep
Adding modernized @wouter 's TrackMaterialParametrizer
which is used to get the parameterised scatters. Now it also works with DDHEP.
Also adds fixes in not very logical choices of including states/fitnodes in TrackMasterFitter
for different track types.
Goes with Moore!3259 (merged) Detector!586 (merged)
Merge request reports
Activity
mentioned in merge request !3686
added RTA label
mentioned in issue Moore#724
added 71 commits
-
52f4b733...f0cbea82 - 69 commits from branch
master
- c784750a - add TrackMaterialParametrizer squashed
- 9c48ebd3 - add LongMuon tracks to clone killing
-
52f4b733...f0cbea82 - 69 commits from branch
mentioned in merge request Moore!3259 (merged)
- Resolved by Maarten Van Veghel
@mveghel I am undrafting this. I will need to have a fresh look to see what I'm missing. If, by chance, anyone can review I would be happy, of course, but I guess that this MR is not of very high priority compared to other things. I can get back to this next week.
I added tests so that they write values of parameterized scatters directly to the reference. If running over a large number of events, it will produce nice and precise numbers there.
Edited by Andrii Usachov
- Resolved by Andrii Usachov
requested review from @mveghel
requested review from @wouter
added 158 commits
-
46d2d432...7d163bde - 155 commits from branch
2024-patches
- f304caba - add TrackMaterialParametrizer squashed
- 926e2405 - add LongMuon tracks to clone killing
- 758c3da7 - Fixed formatting
Toggle commit list-
46d2d432...7d163bde - 155 commits from branch
added 15 commits
-
758c3da7...abef63f6 - 12 commits from branch
2024-patches
- 93722e7a - add TrackMaterialParametrizer squashed
- f25c52d5 - add LongMuon tracks to clone killing
- 02a0e8ac - Fixed formatting
Toggle commit list-
758c3da7...abef63f6 - 12 commits from branch
- Resolved by Andrii Usachov
@ausachov What is the status here? Is this waiting for a further review / development / ready to be tested?
- Resolved by Andrii Usachov