Skip to content

[ML] Data Visualizer: Improves display of long fields values in top values list#241006

Merged
KodeRad merged 7 commits intoelastic:mainfrom
KodeRad:ml-data-visualizer-top-values-truncate
Nov 5, 2025
Merged

[ML] Data Visualizer: Improves display of long fields values in top values list#241006
KodeRad merged 7 commits intoelastic:mainfrom
KodeRad:ml-data-visualizer-top-values-truncate

Conversation

@KodeRad
Copy link
Contributor

@KodeRad KodeRad commented Oct 28, 2025

Summary

Fixes #213732

This PR:

  • adds a truncation to the top values,
  • adjust size of Maps and Distribution charts

Changes affect 3 views:

  • Discover -> Field Statistics
  • Machine Learning -> Data Visualizer
  • Machine Learning -> File Upload
Screenshot 2025-11-03 at 16 15 08

Checklist

Check the PR satisfies following conditions.

Reviewers should verify this PR satisfies this list as well.

  • The PR description includes the appropriate Release Notes section, and the correct release_note:* label is applied per the guidelines
  • Review the backport guidelines and apply applicable backport:* labels.
@KodeRad KodeRad requested a review from peteharverson November 3, 2025 15:11
@KodeRad KodeRad self-assigned this Nov 3, 2025
@KodeRad KodeRad added release_note:enhancement :ml backport:skip This PR does not require backporting Team:ML Team label for ML (also use :ml) t// labels Nov 3, 2025
@KodeRad KodeRad requested a review from jgowdyelastic November 3, 2025 15:19
@KodeRad KodeRad marked this pull request as ready for review November 3, 2025 15:19
@KodeRad KodeRad requested a review from a team as a code owner November 3, 2025 15:19
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

@jgowdyelastic jgowdyelastic changed the title Ml data visualizer top values truncate Nov 3, 2025
Copy link
Member

@jgowdyelastic jgowdyelastic left a comment

Choose a reason for hiding this comment

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

LGTM

@peteharverson peteharverson changed the title [ML] data visualizer top values truncate Nov 5, 2025
Copy link
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.

Tested latest changes and LGTM

@KodeRad KodeRad enabled auto-merge (squash) November 5, 2025 10:39
@KodeRad KodeRad merged commit cea805c into elastic:main Nov 5, 2025
12 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #94 / Dataset Quality Dataset quality handles user privileges "before all" hook in "Dataset quality handles user privileges"
  • [job] [logs] FTR Configs #67 / serverless security UI Security ML Trained models list page navigation renders trained models list

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
dataVisualizer 601.9KB 602.1KB +212.0B

History

cc @KodeRad

wildemat pushed a commit to wildemat/kibana that referenced this pull request Nov 5, 2025
…alues list (elastic#241006)

## Summary

Fixes  elastic#213732

This PR:
- adds a truncation to the top values,
- adjust size of Maps and Distribution charts

Changes affect 3 views:
- Discover -> Field Statistics
- Machine Learning -> Data Visualizer
- Machine Learning -> File Upload

<img width="1919" height="1241" alt="Screenshot 2025-11-03 at 16 15 08"
src="https://github.com/user-attachments/assets/cbd9e5e6-9546-4de6-9bd4-ba6d8d608e2c"
/>



### Checklist

Check the PR satisfies following conditions. 

Reviewers should verify this PR satisfies this list as well.

- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.
viduni94 pushed a commit to viduni94/kibana that referenced this pull request Nov 5, 2025
…alues list (elastic#241006)

## Summary

Fixes  elastic#213732

This PR:
- adds a truncation to the top values,
- adjust size of Maps and Distribution charts

Changes affect 3 views:
- Discover -> Field Statistics
- Machine Learning -> Data Visualizer
- Machine Learning -> File Upload

<img width="1919" height="1241" alt="Screenshot 2025-11-03 at 16 15 08"
src="https://github.com/user-attachments/assets/cbd9e5e6-9546-4de6-9bd4-ba6d8d608e2c"
/>



### Checklist

Check the PR satisfies following conditions. 

Reviewers should verify this PR satisfies this list as well.

- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.
eokoneyo pushed a commit to eokoneyo/kibana that referenced this pull request Dec 2, 2025
…alues list (elastic#241006)

## Summary

Fixes  elastic#213732

This PR:
- adds a truncation to the top values,
- adjust size of Maps and Distribution charts

Changes affect 3 views:
- Discover -> Field Statistics
- Machine Learning -> Data Visualizer
- Machine Learning -> File Upload

<img width="1919" height="1241" alt="Screenshot 2025-11-03 at 16 15 08"
src="https://github.com/user-attachments/assets/cbd9e5e6-9546-4de6-9bd4-ba6d8d608e2c"
/>



### Checklist

Check the PR satisfies following conditions. 

Reviewers should verify this PR satisfies this list as well.

- [x] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)
- [x] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting :ml release_note:enhancement Team:ML Team label for ML (also use :ml) t// v9.3.0

5 participants