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

Size of bubbles not respected on scatter chart #22929

Closed
paoliniluis opened this issue May 24, 2022 · 1 comment
Closed

Size of bubbles not respected on scatter chart #22929

paoliniluis opened this issue May 24, 2022 · 1 comment
Assignees
Labels
.Correctness .Frontend Priority:P2 Average run of the mill bug .Regression Bugs that were previously fixed and/or bugs unintentionally shipped with new features. .Reproduced Issues reproduced in test (usually Cypress) Type:Bug Product defects Visualization/Charts Line, area, bar, combo, and scatter charts.
Milestone

Comments

@paoliniluis
Copy link
Contributor

Describe the bug
If you have a simple query like

select 1 as size, 1 as x, 5 as y union all
select 10 as size, 2 as x, 5 as y union all
select 100 as size, 1 as x, 10 as y union all
select 1000 as size, 2 as x, 10 as y

then the size of the bubbles will be always the same, no matter the value you use in the bubble size config

Logs
NA

To Reproduce

  1. New -> question -> SQL question
  2. use the query above
  3. select a scatter chart and config the chart according to the columns
  4. see the error

Expected behavior
Bubble should have the size of the column "size", not the one from the column "y"

Screenshots
image

Information about your Metabase Installation:

  • Your browser and the version: Brave latest
  • Your operating system: Pop-os 22.04
  • Your databases: Postgres 14.2
  • Metabase version: 1.43.1
  • Metabase hosting environment: Docker
  • Metabase internal database: Postgres 14.2

Severity
P3

Additional context
Reported by a customer

@paoliniluis paoliniluis added Type:Bug Product defects Priority:P3 Cosmetic bugs, minor bugs with a clear workaround Visualization/Charts Line, area, bar, combo, and scatter charts. labels May 24, 2022
@flamber flamber added Priority:P2 Average run of the mill bug .Frontend .Regression Bugs that were previously fixed and/or bugs unintentionally shipped with new features. .Correctness and removed Priority:P3 Cosmetic bugs, minor bugs with a clear workaround labels May 25, 2022
@flamber
Copy link
Contributor

flamber commented May 25, 2022

Regression since 0.43.0, similar to #22527, fixed by #22866.
image

@flamber flamber closed this as completed May 25, 2022
@flamber flamber added this to the 0.43.2 milestone May 25, 2022
@nemanjaglumac nemanjaglumac added this to Backlog in Cypress Testing via automation May 25, 2022
@nemanjaglumac nemanjaglumac moved this from Backlog to To Do in Cypress Testing May 25, 2022
nemanjaglumac added a commit that referenced this issue May 25, 2022
@nemanjaglumac nemanjaglumac added the .Reproduced Issues reproduced in test (usually Cypress) label May 25, 2022
@nemanjaglumac nemanjaglumac removed this from To Do in Cypress Testing May 25, 2022
diogormendes pushed a commit that referenced this issue May 25, 2022
…22950)

Co-authored-by: Nemanja Glumac <31325167+nemanjaglumac@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
.Correctness .Frontend Priority:P2 Average run of the mill bug .Regression Bugs that were previously fixed and/or bugs unintentionally shipped with new features. .Reproduced Issues reproduced in test (usually Cypress) Type:Bug Product defects Visualization/Charts Line, area, bar, combo, and scatter charts.
Projects
None yet
Development

No branches or pull requests

4 participants