-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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 exploration: ensure training filters work as expected #96500
[ML] Data Frame Analytics exploration: ensure training filters work as expected #96500
Conversation
Pinging @elastic/ml-ui (:ml) |
Tested and LGTM 🎉 |
), | ||
}, | ||
], | ||
columnId: `${resultsField}.is_training`, |
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.
do we need to apply an encoding in case the resultsFields
contains special characters?
Testing this I noticed there seems to be an issue with the query to load the evaluate data, in that it doesn't pass through the name of the Looks like the Could this be included as part of this PR? Thanks for taking a look @peteharverson 🙏 Took a quick look and I'm not sure why this is happening. As it is unrelated to the changes in the PR I went ahead and created a separate bug issue for this #96603 so it can be addressed in a separate PR. |
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.
Behavior of Testing / Training filter LGTM.
Makes more sense to fix the issue with the ROC curve and top_classes_field
in a separate PR.
8e24c2c
to
8aa0922
Compare
@elasticmachine merge upstream |
69f5fae
to
77233dc
Compare
💚 Build Succeeded
Metrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: |
Summary
Fixes #96072
Ensures training quick filters work when results field name is not the default.
Checklist
Delete any items that are not applicable to this PR.