-
Notifications
You must be signed in to change notification settings - Fork 36
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
Export results transformation (subtext and hashed) #582
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #582 +/- ##
==========================================
- Coverage 90.15% 90.12% -0.04%
==========================================
Files 129 131 +2
Lines 4642 4708 +66
Branches 752 759 +7
==========================================
+ Hits 4185 4243 +58
- Misses 304 310 +6
- Partials 153 155 +2 ☔ View full report in Codecov by Sentry. |
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.
Implementation is rather tangled. The solution for a quite rare task is not worth the overcomplication of this part of the code.
f2344ae
to
d26ed82
Compare
--subtext reduces json report size of unfiltered data for ML train. |
Description
Please include a summary of the change and which is fixed.
--subtex
to shrink long lines in report and prevent extra memory usage during json load--hashed
to hide sensible information in reportHow has this been tested?
Please describe the tests that you ran to verify your changes.