Skip to content

[PWGEM,Tools] Update EMCal Conversion ML Model and enable batching of… - #17768

Draft
mhemmer-cern wants to merge 3 commits into
AliceO2Group:masterfrom
mhemmer-cern:emcal-ml
Draft

[PWGEM,Tools] Update EMCal Conversion ML Model and enable batching of…#17768
mhemmer-cern wants to merge 3 commits into
AliceO2Group:masterfrom
mhemmer-cern:emcal-ml

Conversation

@mhemmer-cern

Copy link
Copy Markdown
Collaborator

… ml models

  • Add option to use ml models in batches, instead of testing one candidate per call to reduce speed up application of ml models
  • Update emcalPhotonMcTask.cxx to incoporate these changes
  • Update histograms that will be stored and filled

… ml models

- Add option to use ml models in batches, instead of testing one candidate per call to reduce speed up application of ml models
- Update `emcalPhotonMcTask.cxx` to incoporate these changes
- Update histograms that will be stored and filled
@github-actions github-actions Bot added pwgem photon Photon and neutral meson PAG label tools labels Sep 4, 2026
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

O2 linter results: ❌ 0 errors, ⚠️ 0 warnings, 🔕 2 disabled

@mhemmer-cern mhemmer-cern changed the title [PWGEM,TOOLS] Update EMCal Conversion ML Model and enable batching of… [PWGEM, Tools] Update EMCal Conversion ML Model and enable batching of… Sep 4, 2026
@mhemmer-cern mhemmer-cern changed the title [PWGEM, Tools] Update EMCal Conversion ML Model and enable batching of… [PWGEM,Tools] Update EMCal Conversion ML Model and enable batching of… Sep 4, 2026
@mhemmer-cern

Copy link
Copy Markdown
Collaborator Author

Hello @fcatalan92 @fmazzasc
While working on some ML model for EMCal cluster, I tried adding the option to use the ml output in batches. Also looking into model.h it looked to me like there is a potential memory issue where a pointer gets created pointing to a local variable inside the function. After return is called the memory for the variable should be freed making the pointer point to garbage.
Could you please have a look into the changes and the comment made by this commit to the ML code?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

photon Photon and neutral meson PAG label pwgem tools

Development

Successfully merging this pull request may close these issues.

1 participant