Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Bug fix in skimmed data (dca for combined tracks) + extension adding number of shared clusters monitoring #13173

Open
miranov25 opened this issue May 29, 2024 · 0 comments
Open

Bug fix in skimmed data (dca for combined tracks) + extension adding number of shared clusters monitoring #13173

miranov25 opened this issue May 29, 2024 · 0 comments

Comments

@miranov25
Copy link
Contributor

miranov25 commented May 29, 2024 •
edited
Loading

Relates to - #12928

1.) DCA for combined tracks is stored relative to (0,0). It can be recalculated using the vertex position, but some precision is lost in this process.

2.) The number of shared clusters is heavily dependent on the error parameterization and there are indications of possible bug in sector/stack merging, as evidenced by a distribution with a stack-like structure.

In figure below the distribution of the shared clusters for old(blue) and new (red) error parameterization. In new error parameterization it is assumed that amount of the shared clusters will be reduced.

image
miranov25 pushed a commit to miranov25/AliceO2 that referenced this issue May 29, 2024
AliceO2Group#13173 - fixing DCA
ac2565a
@miranov25 miranov25 mentioned this issue May 29, 2024
Merged
miranov25 pushed a commit to miranov25/AliceO2 that referenced this issue May 29, 2024
AliceO2Group#13173 - using clang-format
8d0e51a
davidrohr pushed a commit that referenced this issue May 29, 2024
@miranov25
#13173 - fixing DCA for combined tracking + adding nshared clusters (#…
877eba6 
…13176)

* #13173 - fixing DCA

* #13173 - using clang-format

---------

Co-authored-by: miranov25 <[email protected]>
benedikt-voelkel pushed a commit that referenced this issue Jun 6, 2024
@miranov25
#13173 - fixing DCA for combined tracking + adding nshared clusters (#…
76880b8 
…13176)

* #13173 - fixing DCA

* #13173 - using clang-format

---------

Co-authored-by: miranov25 <[email protected]>
(cherry picked from commit 877eba6)
chiarazampolli pushed a commit that referenced this issue Jun 7, 2024
@miranov25 @chiarazampolli
#13173 - fixing DCA for combined tracking + adding nshared clusters (#…
82bfcec 
…13176)

* #13173 - fixing DCA

* #13173 - using clang-format

---------

Co-authored-by: miranov25 <[email protected]>
(cherry picked from commit 877eba6)
benedikt-voelkel pushed a commit that referenced this issue Jun 7, 2024
@miranov25
#13173 - fixing DCA for combined tracking + adding nshared clusters (#…
00f1d30 
…13176)

* #13173 - fixing DCA

* #13173 - using clang-format

---------

Co-authored-by: miranov25 <[email protected]>
(cherry picked from commit 877eba6)
benedikt-voelkel pushed a commit that referenced this issue Jun 21, 2024
@miranov25
#13173 - fixing DCA for combined tracking + adding nshared clusters (#…
3310da8 
…13176)

* #13173 - fixing DCA

* #13173 - using clang-format

---------

Co-authored-by: miranov25 <[email protected]>
(cherry picked from commit 877eba6)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@miranov25