Skip to content
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] Index data visualizer - wrong initial label on sampling percentage slider #146323

Closed
pheyos opened this issue Nov 24, 2022 · 1 comment · Fixed by #150389
Closed

[ML] Index data visualizer - wrong initial label on sampling percentage slider #146323

pheyos opened this issue Nov 24, 2022 · 1 comment · Fixed by #150389
Assignees
Labels
bug Fixes for quality problems that affect the customer experience cosmetic Feature:File and Index Data Viz ML file and index data visualizer :ml v8.7.0

Comments

@pheyos
Copy link
Member

pheyos commented Nov 24, 2022

Found in version

  • 8.6.0 BC2 (but might have been like this in earlier versions as well)

Browser

  • Chrome

Steps to reproduce

  • Open an index in the data visualizer
  • In the random sampler popover switch to On - manual

Expected result

  • The Sampling percentage is initially set to 50% and the label of the slider shows 50

Actual result

  • The Sampling percentage is initially set to 50% and the label of the slider shows 100:
    image

Additional information

  • When moving the slider back and forth again, it displays the expected 50.
@elasticmachine
Copy link
Contributor

Pinging @elastic/ml-ui (:ml)

@peteharverson peteharverson added the Feature:File and Index Data Viz ML file and index data visualizer label Jan 24, 2023
@peteharverson peteharverson added the bug Fixes for quality problems that affect the customer experience label Feb 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience cosmetic Feature:File and Index Data Viz ML file and index data visualizer :ml v8.7.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants