Momentum resolution study lingzhu
This MR uses a different parameterization for momentum and improves the momentum resolution.
The short of it is: use dev_velo_kalman_endvelo_states
as the input for velo state and the state at z_mid_t
as input for SciFi state. The performance(momentum resolution and throughput) of this new parameterization can be seen in this report on WP2 meeting
- The origin qop calculation in
lf_quality_filter
and the related information(parameters, states, tracks) are gone - The new qop calculation is done in
scifi_consolidate_tracks
this branch relies on end_velo_state_maxime
Closes #126 (closed)
Edited by Dorothea Vom Bruch
Merge request reports
Activity
added 1 commit
- 6bff186e - modify .gitlab-ci.yml and test sequence hlt1_pp_scifi_v6
added 148 commits
-
6bff186e...7d59cff2 - 130 commits from branch
master
- e0ef36b2 - Added the state at the end of the velo
- 69c20034 - Adapt PV logic to Kalman filter state calculation.
- b82e443c - Significant refactor of states.
- 2b3ef06f - Fixed formatting
- 9ec86331 - Bugfix of pv beamline extrapolate.
- ebf56c7a - Added beamline position dependence
- d80b5ac4 - Update the rest of the covariance matrix
- 22b59649 - Changed least square fit to linear fit between first and last hit as starting point for kalman fit
- 5cd914b1 - Added dependance to the dev_beamline position defined in constants
- 3213a01c - Removed warning.
- 968c51f9 - Change which state is stored in the host_buffer
- e0a32485 - Save beamline and endvelo state to hostbuffer and fix the translation to V2 tracks
- 60c69aeb - Fix translation to V2 tracks
- b5fd572d - Fix error in covariance matrix transportation
- c9905c08 - Calculate lms fit beamline state and use it in CompassUT, which is optimized for that state.
- 5bb30f9c - Fixed formatting
- 51229851 - Fixed wrong comment
- 2a62dfdd - Merge remote-tracking branch 'origin/end_velo_state_maxime' into momentum_resolution_study_lingzhu
Toggle commit list-
6bff186e...7d59cff2 - 130 commits from branch
- Resolved by Lingzhu Bian
- Resolved by Dorothea Vom Bruch
- Resolved by Lingzhu Bian
added 1 commit
- ab3656be - switch back to the default sequence for the CI tests and delete the qop...
added 1 commit
- 60008042 - put qop calculation into a separate __device__ function
- Resolved by Dorothea Vom Bruch
@lbian can you please add to the MR description a link to your nice presentation in the WP2 meeting this Monday? That summarizes nicely the performance improvements.
Please register or sign in to reply