Skip to content

[ML] Fixes chart in single metric anomaly detection wizard#214837

Merged
jgowdyelastic merged 2 commits intoelastic:mainfrom
jgowdyelastic:fixing-single-metric-wizard-anomaly-results
Mar 18, 2025
Merged

[ML] Fixes chart in single metric anomaly detection wizard#214837
jgowdyelastic merged 2 commits intoelastic:mainfrom
jgowdyelastic:fixing-single-metric-wizard-anomaly-results

Conversation

@jgowdyelastic
Copy link
Copy Markdown
Member

@jgowdyelastic jgowdyelastic commented Mar 17, 2025

Fixes an issue with the single metric wizard chart where an error is thrown when attempting to adjust the buckets of the previous batch of loaded results.

Before

image
image

After
image

@jgowdyelastic jgowdyelastic self-assigned this Mar 17, 2025
@jgowdyelastic jgowdyelastic marked this pull request as ready for review March 17, 2025 17:15
@jgowdyelastic jgowdyelastic requested a review from a team as a code owner March 17, 2025 17:15
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui (:ml)

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
ml 5.4MB 5.4MB +9.0B

cc @jgowdyelastic

@peteharverson peteharverson changed the title [ML] Fixing single metric wizard anomaly results Mar 18, 2025
Copy link
Copy Markdown
Contributor

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Tested and confirmed the model bounds are now plotted correctly.

@jgowdyelastic jgowdyelastic merged commit d10e1f8 into elastic:main Mar 18, 2025
10 checks passed
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/13927056689

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Mar 18, 2025
…14837)

Fixes an issue with the single metric wizard chart where an error is
thrown when attempting to adjust the buckets of the previous batch of
loaded results.

**Before**

![image](https://github.com/user-attachments/assets/ac9feef5-f354-4423-b34d-39afc029f024)

![image](https://github.com/user-attachments/assets/7d62db99-e7d9-4a08-a823-fa4a4a5e79af)

**After**

![image](https://github.com/user-attachments/assets/a3d9c3fd-3eac-4b82-b9d6-319735258661)

(cherry picked from commit d10e1f8)
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Mar 19, 2025
…4837) (#215000)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[ML] Fixes chart in single metric anomaly detection wizard
(#214837)](#214837)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"James
Gowdy","email":"jgowdy@elastic.co"},"sourceCommit":{"committedDate":"2025-03-18T15:25:31Z","message":"[ML]
Fixes chart in single metric anomaly detection wizard (#214837)\n\nFixes
an issue with the single metric wizard chart where an error is\nthrown
when attempting to adjust the buckets of the previous batch of\nloaded
results.\n\n**Before**\n\n\n![image](https://github.com/user-attachments/assets/ac9feef5-f354-4423-b34d-39afc029f024)\n\n![image](https://github.com/user-attachments/assets/7d62db99-e7d9-4a08-a823-fa4a4a5e79af)\n\n\n\n**After**\n\n![image](https://github.com/user-attachments/assets/a3d9c3fd-3eac-4b82-b9d6-319735258661)","sha":"d10e1f8f17b110d7831670ba6f16bcd78ab57974","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix",":ml","Feature:Anomaly
Detection","backport:version","v9.1.0","v8.19.0"],"title":"[ML] Fixes
chart in single metric anomaly detection
wizard","number":214837,"url":"https://github.com/elastic/kibana/pull/214837","mergeCommit":{"message":"[ML]
Fixes chart in single metric anomaly detection wizard (#214837)\n\nFixes
an issue with the single metric wizard chart where an error is\nthrown
when attempting to adjust the buckets of the previous batch of\nloaded
results.\n\n**Before**\n\n\n![image](https://github.com/user-attachments/assets/ac9feef5-f354-4423-b34d-39afc029f024)\n\n![image](https://github.com/user-attachments/assets/7d62db99-e7d9-4a08-a823-fa4a4a5e79af)\n\n\n\n**After**\n\n![image](https://github.com/user-attachments/assets/a3d9c3fd-3eac-4b82-b9d6-319735258661)","sha":"d10e1f8f17b110d7831670ba6f16bcd78ab57974"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/214837","number":214837,"mergeCommit":{"message":"[ML]
Fixes chart in single metric anomaly detection wizard (#214837)\n\nFixes
an issue with the single metric wizard chart where an error is\nthrown
when attempting to adjust the buckets of the previous batch of\nloaded
results.\n\n**Before**\n\n\n![image](https://github.com/user-attachments/assets/ac9feef5-f354-4423-b34d-39afc029f024)\n\n![image](https://github.com/user-attachments/assets/7d62db99-e7d9-4a08-a823-fa4a4a5e79af)\n\n\n\n**After**\n\n![image](https://github.com/user-attachments/assets/a3d9c3fd-3eac-4b82-b9d6-319735258661)","sha":"d10e1f8f17b110d7831670ba6f16bcd78ab57974"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: James Gowdy <jgowdy@elastic.co>
clintandrewhall pushed a commit to clintandrewhall/kibana that referenced this pull request Mar 20, 2025
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Mar 22, 2025
cqliu1 pushed a commit to cqliu1/kibana that referenced this pull request Mar 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4 participants