Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Demonstration of ML Integration in EICrecon #1340

Closed
ruse-traveler opened this issue Mar 20, 2024 · 1 comment · Fixed by #1358
Closed

Demonstration of ML Integration in EICrecon #1340

ruse-traveler opened this issue Mar 20, 2024 · 1 comment · Fixed by #1358
Labels
help wanted Extra attention is needed TDR:Calorimetry Relevant to calorimetry TDR wants/needs topic: AI/ML Issues pertinent to AI and machine learning topic: documentation Improvements or additions to documentation topic: reconstruction

Comments

@ruse-traveler
Copy link
Contributor

As of eic/containers#9 , TMVA::SOFIE is now included as part of eic-shell for ONNX support. However, as of the submitting of this issue, there are no examples in EICrecon/main demonstrating how to use this functionality.

Describe the solution you'd like
There have been several standalone studies of ePIC simulations which utilized ML in some capacity (e.g see the ePIC AI/ML Town Hall or AI4EIC 2023), including many which would be good additions to our suite of reconstruction algorithms.

Additional Context
Support for ONNX/examples of how to use this in EICrecon was identified as one of the TDR wants from the calorimetry side.

@ruse-traveler ruse-traveler added topic: documentation Improvements or additions to documentation help wanted Extra attention is needed topic: reconstruction TDR:Calorimetry Relevant to calorimetry TDR wants/needs topic: AI/ML Issues pertinent to AI and machine learning labels Mar 20, 2024
@veprbl
Copy link
Member

veprbl commented Mar 21, 2024

We still don't have onnxruntime for inference part https://eicweb.phy.anl.gov/containers/eic_container/-/merge_requests/789
It's not clear (at least, to me) if TMVA is a good candidate for inference over PODIO objects. There was also some nice discussion about EDM changes needed to support consistent inputs between inference and training as well a report the correct labels https://chat.epic-eic.org/main/pl/hi7hf5dcepft9m9wsbd1fgiyzc

There is some work to be done on the training side. The idea is that, since we don't have a conditions database, we are using artifacts (e.g. stored in https://github.com/eic/epic-data) defined in epic geometry compact XML files. Those artifacts need to be reproducible to prevent black boxing. We want to have this working via CI, and work was started on this in eic/detector_benchmarks#10 .

@veprbl veprbl linked a pull request Apr 2, 2024 that will close this issue
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed TDR:Calorimetry Relevant to calorimetry TDR wants/needs topic: AI/ML Issues pertinent to AI and machine learning topic: documentation Improvements or additions to documentation topic: reconstruction
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants