Skip to content

Conversation

@lmoneta
Copy link
Member

@lmoneta lmoneta commented Jan 23, 2026

No description provided.

@lmoneta lmoneta requested a review from couet as a code owner January 23, 2026 18:28
@lmoneta lmoneta force-pushed the tmva_sofie_new_onnx_tutorial branch from 97c6d26 to f0e43bd Compare January 23, 2026 22:14
@github-actions
Copy link

github-actions bot commented Jan 24, 2026

Test Results

    22 files      22 suites   3d 18h 43m 27s ⏱️
 3 771 tests  3 771 ✅ 0 💤 0 ❌
75 890 runs  75 890 ✅ 0 💤 0 ❌

Results for commit f34b225.

♻️ This comment has been updated with latest results.

@lmoneta lmoneta force-pushed the tmva_sofie_new_onnx_tutorial branch from f0e43bd to caca586 Compare January 24, 2026 22:37
@lmoneta lmoneta requested a review from sanjibansg January 26, 2026 07:06
@lmoneta lmoneta force-pushed the tmva_sofie_new_onnx_tutorial branch from 940632f to d7b0912 Compare January 26, 2026 08:30
…rch or onnx are found

Also in case of failure calling export , exit tutorials without giving an error

Improve also comments following Sanjiban suggested review
@lmoneta lmoneta force-pushed the tmva_sofie_new_onnx_tutorial branch from d7b0912 to f34b225 Compare January 26, 2026 10:14
@lmoneta lmoneta requested a review from bellenot as a code owner January 26, 2026 10:14
Copy link
Contributor

@sanjibansg sanjibansg left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks!

@lmoneta lmoneta merged commit a107e6b into root-project:master Jan 27, 2026
29 of 30 checks passed
@lmoneta lmoneta deleted the tmva_sofie_new_onnx_tutorial branch January 27, 2026 09:57
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