-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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] Data Frame Analytics results: ensure model evaluation stats are shown #97486
[ML] Data Frame Analytics results: ensure model evaluation stats are shown #97486
Conversation
Pinging @elastic/ml-ui (:ml) |
💚 Build SucceededMetrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: |
This is ready for a look when you get a chance! 🙏 cc @qn895, @peteharverson, @darnautov |
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. Could this also be backported for 7.12.2
?
Will be backported to 7.12.2 👍 |
…shown (elastic#97486) * ensure we check for NaN and Infinity in eval response * add unit test
…shown (elastic#97486) * ensure we check for NaN and Infinity in eval response * add unit test
…shown (elastic#97486) * ensure we check for NaN and Infinity in eval response * add unit test
Summary
Fixes #97322
This PR ensures that regression evaluation results handle
NaN
andInfinity
values returned from_evaluate
endpoint.A unit test for
getValuesFromResponse
has been added as well.Before:
After:
Checklist
Delete any items that are not applicable to this PR.