-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[ML] Fixing overflowing metric values in field cards #29255
[ML] Fixing overflowing metric values in field cards #29255
Conversation
x-pack/plugins/ml/public/file_datavisualizer/components/fields_stats/fields_stats.js
Outdated
Show resolved
Hide resolved
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.
LGTM, just added a comment on a possible helper function.
💔 Build Failed |
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.
LGTM ⚡️
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.
LGTM
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.
LGTM
💔 Build Failed |
retest |
💔 Build Failed |
retest |
💔 Build Failed |
retest |
4e566bf
to
d680de1
Compare
💚 Build Succeeded |
* [ML] Fixing overflowing metric values in field cards * adding better rounding for tiny numbers * removing debug line
Round min, max and median values to 2dp to fix occasional overflow.
Before:
After:
For very small numbers which would normally round to
0
, a new function converts them to index notation.e.g.
-0.000000234
is converted to-2.34e-7
Adds a new formatter function to do the rounding along with some tests.
Fixes #28567