-
Notifications
You must be signed in to change notification settings - Fork 157
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
ci: Create job summaries in python #2339
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2339 +/- ##
=======================================
Coverage 49.66% 49.66%
=======================================
Files 453 453
Lines 25533 25533
Branches 11706 11706
=======================================
Hits 12680 12680
Misses 4574 4574
Partials 8279 8279 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
📊 Physics performance monitoring for 40b6fd5Summary VertexingSeedingCKFAmbiguity resolutionTruth tracking (Kalman Filter)Truth tracking (GSF) |
The texme header doesn't work as well as I thought it would, and also the links don't currently work in the job summary (I have an idea how to fix that) so I'll switch to just using multiple separate outputs from python |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice!
Ok, I added some functionality to herald to redirect to an artifact from a run id while maintaining paths. Now the physmon summary markdown should contain working links. |
This PR changes the physmon report CI job to basically pipe through the `summary.md` that we create inside the physmon job to the comment. This should be more robust than what we had before. Blocked by - #2339
No description provided.