-
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] Anomaly Explorer charts #21163
Labels
Comments
Pinging @elastic/ml-ui |
walterra
added a commit
that referenced
this issue
Sep 7, 2018
After the migration to React in #22622, this PR adds more test coverage based on jest to the Anomaly Explorer charts. This is done as preparation for some further refactoring (#22626) and the integration of the support for charts for the rare detector (#21163). The tests use mock data based on a standard multi-metric job using the farequote dataset. Besided the tests this PR includes the following changes: - In explore_series.js the path is no longer a concatenanted string but now using a template literal - The exploreSeries() function is no longer calling window.open by itself, it just returns the link because of the above, renamed exploreSeries() to getExploreSeriesLink() and moved it to chart_utils.js - explorer_charts_container_service.js is no longer requiring jQuery itself, it now receiving the required element as a factory argument. Further work on this is planned to get rid of jQuery.
walterra
added a commit
to walterra/kibana
that referenced
this issue
Sep 7, 2018
After the migration to React in elastic#22622, this PR adds more test coverage based on jest to the Anomaly Explorer charts. This is done as preparation for some further refactoring (elastic#22626) and the integration of the support for charts for the rare detector (elastic#21163). The tests use mock data based on a standard multi-metric job using the farequote dataset. Besided the tests this PR includes the following changes: - In explore_series.js the path is no longer a concatenanted string but now using a template literal - The exploreSeries() function is no longer calling window.open by itself, it just returns the link because of the above, renamed exploreSeries() to getExploreSeriesLink() and moved it to chart_utils.js - explorer_charts_container_service.js is no longer requiring jQuery itself, it now receiving the required element as a factory argument. Further work on this is planned to get rid of jQuery.
walterra
added a commit
that referenced
this issue
Sep 7, 2018
After the migration to React in #22622, this PR adds more test coverage based on jest to the Anomaly Explorer charts. This is done as preparation for some further refactoring (#22626) and the integration of the support for charts for the rare detector (#21163). The tests use mock data based on a standard multi-metric job using the farequote dataset. Besided the tests this PR includes the following changes: - In explore_series.js the path is no longer a concatenanted string but now using a template literal - The exploreSeries() function is no longer calling window.open by itself, it just returns the link because of the above, renamed exploreSeries() to getExploreSeriesLink() and moved it to chart_utils.js - explorer_charts_container_service.js is no longer requiring jQuery itself, it now receiving the required element as a factory argument. Further work on this is planned to get rid of jQuery.
61 tasks
walterra
changed the title
[ML] Anomaly explorer chart for rare detector
[ML] Anomaly Explorer charts for rare detector and population analysis.
Oct 1, 2018
This was referenced Oct 30, 2018
2 tasks
sophiec20
added
Feature:Anomaly Detection
ML anomaly detection
and removed
Feature:ml-results
legacy - do not use
labels
Jun 19, 2019
walterra
changed the title
[ML] Anomaly Explorer charts for rare detector and population analysis.
[ML] Anomaly Explorer charts
Jan 26, 2021
Merged
2 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Tracking issue for the implementation of a custom charts for detector functions
rare
,over
,lat_long
.Investigate
Implementation
Phase 1 #23423
results
length is 0master
even without PR 1 so not a problem introduced by the rare chart itself. 2f75c7fover
andby
, theby
field can have an influencer on the outcome and a total high count could be less anomalous than a lower count one.undefined
in tooltip [ML] Anomaly Explorer Rare/Population Charts #23423 (comment) 5f5ae7aPhase 2
over
fields — 33cbdb3count
andsum
based aggregations. — [ML] Improve sampling and normalization of population chart. #24402SAMPLER_TOP_TERMS_SHARD_SIZE
a function argument [ML] Anomaly Explorer Rare/Population Charts #23423 (comment)Phase 3 / future work / nice to have
freq_rare
https://www.elastic.co/guide/en/x-pack/current/ml-rare-functions.html#ml-freq-rarerare/freq_rare
plusover
fieldsEuiDescriptionList
instead ofmlDescriptionList
styles. [ML] Anomaly Explorer Rare/Population Charts #23423 (comment)function
does not map to an ES aggregation (e.g. varp, info_content) if model plot is enabled. Currently a metric style chart is displayed in these cases.Part of #22922.
The text was updated successfully, but these errors were encountered: