Skip to content

Add new process to apply TRestDataSetGainMap to track line - #53

Draft
AlvaroEzq wants to merge 4 commits into
masterfrom
line_gm_correction
Draft

AlvaroEzq wants to merge 4 commits into
masterfrom
line_gm_correction

Conversation

@AlvaroEzq

@AlvaroEzq AlvaroEzq commented Sep 30, 2025 •

Copy link
Copy Markdown
Contributor

AlvaroEzq Large: 287 Powered by Pull Request Badge

The new process TRestTrackLineGainMapCorrectionProcess applies the calibration of a TRestDataSetGainMap to a TRestTrackEvent which should have been already linearized (by the process TRestTrackLinearizationProcess). The key point of this process is that it applies the gain map hit by hit (where this hits are the nodes of the linearized track).

Note

Suggestions for the process name are welcomed 😅

@AlvaroEzq
AlvaroEzq marked this pull request as draft September 30, 2025 08:58
Comment thread src/TRestTrackLineGainMapCorrectionProcess.cxx Outdated
double zClosestYhit = 9e9;
double eClosestYhit = 0;
double yClosestYhit = 0;
for (int j = 0; j < hitsY->GetNumberOfHits(); j++) {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
for (int j = 0; j < hitsY->GetNumberOfHits(); j++) {
for (size_t j = 0; j < hitsY->GetNumberOfHits(); j++) {

Comment thread src/TRestTrackLineGainMapCorrectionProcess.cxx Outdated
AlvaroEzq and others added 2 commits August 24, 2026 12:10
Co-authored-by: Juan Antonio García Pascual <juanan318@gmail.com>
Co-authored-by: Juan Antonio García Pascual <juanan318@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants