Skip to content

Conversation

@SusanneGlaessel
Copy link
Contributor

The TRD pid is integrated in the following way:

The Pid framework contains two steps for the Tof pid:

  1. Calculating MC-probabilities / fitting
  2. Filling

The integrated Trd pid consists of three steps:
Preparation: Creating MC histograms

  1. Calculating MC-probabilities
  2. Filling

The preparation and step 1) are performed independently of each other for both detector pids.

Step 2) is now combined - the filling of both pids can be done simultaneously. It is still possible to run only one of the detector pids by using the same executable. The selection of what pid to run, is handled by the first argument of the executable (see README.md for more details.).

@lubynets
Copy link
Contributor

lubynets commented Jan 9, 2026

Dear @SusanneGlaessel, thanks for the extensive development!

  1. Please consider my comments which I left in the code.
  2. Remove the files .DS_Store from the PR and add them to .gitignore so git will not include them any more (those which are already included you need to remove from git index manually).
  3. Adapt the clang-format workflow to the changed naming of directories (link), and apply clang-format locally before updating the PR.

Copy link
Contributor

@lubynets lubynets left a comment

Choose a reason for hiding this comment

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

@SusanneGlaessel thanks for addressing my comments!
I still have minor comments/suggestions which I left in the code.

@SusanneGlaessel
Copy link
Contributor Author

Dear @lubynets,
thank you very much for checking the code and for your comments.
I have incorporated now everything, just that one line pid = TMath::Sign(1, charge); I left like it is (see comment). Is it ok for you?
Thank you and kind regards,
Susanne

@lubynets
Copy link
Contributor

@SusanneGlaessel thanks again for the development, and for processing my comments and suggestions!

@lubynets lubynets merged commit c60d6c0 into HeavyIonAnalysis:master Jan 15, 2026
3 checks passed
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