Skip to content

Fix truth primary flag storage type - #275

Merged
jdkio merged 1 commit into
mainfrom
jdkio/fix_truth_info_is_primary_type
Jul 20, 2026
Merged

Fix truth primary flag storage type#275
jdkio merged 1 commit into
mainfrom
jdkio/fix_truth_info_is_primary_type

Conversation

@jdkio

@jdkio jdkio commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

The branch is written as a bool but used int array to fill. This would work but prints a warning. This fixes the issue.

@jdkio jdkio added the bug Something isn't working label Jul 20, 2026
@jdkio

jdkio commented Jul 20, 2026

Copy link
Copy Markdown
Contributor Author

Validated that nothing changes other than the warning:

# Before:
Truth_Info->Scan("RecoTrackPrimaryParticleIsPrimary:RecoTrackSecondaryParticleIsPrimary", "RecoTrackN>0")
***********************************************
*    Row   * Instance * RecoTrack * RecoTrack *
***********************************************
*        2 *        0 *         1 *         1 *
*        3 *        0 *         1 *         0 *
*        3 *        1 *         0 *         0 *
*        5 *        0 *         1 *         0 *
*        9 *        0 *         1 *         1 *
*       12 *        0 *         1 *         1 *
*       14 *        0 *         1 *         0 *
*       18 *        0 *         1 *         1 *
*       19 *        0 *         1 *         0 *
*       23 *        0 *         1 *         0 *
*       30 *        0 *         1 *         0 *
*       31 *        0 *         1 *         1 *
*       32 *        0 *         1 *         0 *
*       38 *        0 *         1 *         0 *
*       40 *        0 *         1 *         0 *
*       44 *        0 *         1 *         1 *
*       44 *        1 *         0 *         0 *
*       45 *        0 *         1 *         1 *
*       46 *        0 *         1 *         1 *
*       47 *        0 *         1 *         0 *
*       53 *        0 *         1 *         1 *
*       54 *        0 *         0 *         1 *
*       56 *        0 *         1 *         1 *
*       59 *        0 *         1 *         0 *
*       60 *        0 *         1 *         0 *

# After:
Truth_Info->Scan("RecoTrackPrimaryParticleIsPrimary:RecoTrackSecondaryParticleIsPrimary", "RecoTrackN>0")
***********************************************
*    Row   * Instance * RecoTrack * RecoTrack *
***********************************************
*        2 *        0 *         1 *         1 *
*        3 *        0 *         1 *         0 *
*        3 *        1 *         1 *         0 *
*        5 *        0 *         1 *         0 *
*        9 *        0 *         1 *         1 *
*       12 *        0 *         1 *         1 *
*       14 *        0 *         1 *         0 *
*       18 *        0 *         1 *         1 *
*       19 *        0 *         1 *         0 *
*       23 *        0 *         1 *         0 *
*       30 *        0 *         1 *         0 *
*       31 *        0 *         1 *         1 *
*       32 *        0 *         1 *         0 *
*       38 *        0 *         1 *         0 *
*       40 *        0 *         1 *         0 *
*       44 *        0 *         1 *         1 *
*       44 *        1 *         1 *         1 *
*       45 *        0 *         1 *         1 *
*       46 *        0 *         1 *         1 *
*       47 *        0 *         1 *         0 *
*       53 *        0 *         1 *         1 *
*       54 *        0 *         0 *         1 *
*       56 *        0 *         1 *         1 *
*       59 *        0 *         1 *         0 *
*       60 *        0 *         1 *         0 *

@jdkio
jdkio merged commit 180759c into main Jul 20, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants