Skip to content

Test onnx retraining and suggestion workflow locally - Do not merge #213

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 43 commits into
base: master
Choose a base branch
from
Open

Test onnx retraining and suggestion workflow locally - Do not merge #213

wants to merge 43 commits into from
+349 −195

Conversation

@simonge
Copy link
Contributor

@simonge simonge commented Oct 24, 2025

Briefly, what does this PR introduce?

Tries to minimize the workflow to test each component of the onnx retaining locally before pushing the changes and testing with the epic detector repository.

simonge
simonge commented Oct 27, 2025
View reviewed changes
Copy link
Contributor Author

@simonge simonge left a comment

Choose a reason for hiding this comment

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

🤖 Automated review with updated calibration URLs calibrations/onnx/Low-Q2_Steering_Reconstruction.onnx for PR #213

benchmarks/lowq2_reconstruction/calibrations.xml
<plugin name="epic_FileLoader">
<arg value="cache:$DETECTOR_PATH:/opt/detector"/>
<arg value="file:calibrations/onnx/Low-Q2_Steering_Reconstruction.onnx"/>
<arg value="url:root://dtn-eic.jlab.org//volatile/eic/EPIC/xrdtest/CALIB/2025/5a202b05d865214e3c399883ee13859318044678f87d7b3d9fa8ff526b4909b6/Low-Q2_Steering_Reconstruction.onnx"/>
Copy link
Contributor Author

@simonge simonge Oct 27, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

🤖 Automated Calibration calibrations/onnx/Low-Q2_Steering_Reconstruction.onnx Update (Updated)

A new calibration has been generated and is ready for use.

Last updated: 2025-10-29 11:49:11 UTC

Suggested change
<arg value="url:root://dtn-eic.jlab.org//volatile/eic/EPIC/xrdtest/CALIB/2025/5a202b05d865214e3c399883ee13859318044678f87d7b3d9fa8ff526b4909b6/Low-Q2_Steering_Reconstruction.onnx"/>
<arg value="url:root://dtn-eic.jlab.org//volatile/eic/EPIC/xrdtest/CALIB//87dd0f9bb8e6dadeef53850550da6f11223213739a011ca79c3ccd56b4b00be2/Low-Q2_Steering_Reconstruction.onnx"/>

@simonge
Copy link
Contributor Author

simonge commented Oct 29, 2025 •
edited
Loading

🤖 Automated Calibration calibrations/onnx/Low-Q2_Steering_Reconstruction.onnx Update (Updated)

A new calibration has been generated and is ready for use.

File: benchmarks/lowq2_reconstruction/calibrations.xml
Line: 15
Last updated: 2025-10-31 22:28:26 UTC

Current line:

<arg value="url:root://dtn-eic.jlab.org//volatile/eic/EPIC/xrdtest/CALIB/2025/5a202b05d865214e3c399883ee13859318044678f87d7b3d9fa8ff526b4909b6/Low-Q2_Steering_Reconstruction.onnx"/>

Proposed change:

<arg value="url:root://dtn-eic.jlab.org//volatile/eic/EPIC/xrdtest/CALIB//ee0b57f256c3d52282dd75d0d704c30609ee9ca8fb5bfdeb74dd140e51356c8b/Low-Q2_Steering_Reconstruction.onnx"/>

Please update the calibration URL in benchmarks/lowq2_reconstruction/calibrations.xml at line 15.

📊 Before Calibration Update

Before Image

✨ After Calibration Update

After Image

📁 For comprehensive results and full-resolution artifacts, please visit: https://eicweb.phy.anl.gov/EIC/benchmarks/detector_benchmarks/-/jobs/6589273/artifacts/browse/results/lowq2_reconstruction/

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@simonge