Skip to content

Charge field#132

Merged
lubynets merged 4 commits intoHeavyIonAnalysis:masterfrom
lubynets:dev
Aug 2, 2024
Merged

Charge field#132
lubynets merged 4 commits intoHeavyIonAnalysis:masterfrom
lubynets:dev

Conversation

@lubynets
Copy link
Copy Markdown
Contributor

@lubynets lubynets commented Aug 2, 2024

  1. Added charge field for Track. For daughter class Particle it works in the same way as mass field - linked to the PDG code unless user explicitly sets specific value different from table one.
  2. In order to avoid a conflict of field names an additional check is implemented - if one tries to add a field with already existing name (both default or user-defined), an error will be thrown.

Note: In some pieces of code based on AnalysisTree (e.g. CbmRoot interface, PFSimple, Pid etc) the charge could be set as a user-defined field. Now it should be fixed, and a default field should be used. Check described in item (2) will help to localize these places.

@lubynets lubynets merged commit 53759d9 into HeavyIonAnalysis:master Aug 2, 2024
@lubynets lubynets deleted the dev branch August 2, 2024 14:53
lubynets added a commit to lubynets/AnalysisTree that referenced this pull request Jul 31, 2025
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.

1 participant