-
Notifications
You must be signed in to change notification settings - Fork 603
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
Upload scrublet scores on test failure #3069
Conversation
ilan-gold
commented
May 17, 2024
•
edited by flying-sheep
Loading
edited by flying-sheep
- Helps debugging Broken Scrublet Test #3068
- Tests included or not required because:
- Release notes not necessary because:
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3069 +/- ##
=======================================
Coverage 76.73% 76.73%
=======================================
Files 109 109
Lines 12529 12529
=======================================
Hits 9614 9614
Misses 2915 2915 |
hmm, we could just make the upload conditional to the file existing and merge this into |
TODO: creation of zarr store conditional on failed test, and then do the upload conditional on file existence |
Co-authored-by: Ilan Gold <[email protected]>