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

Metrics visibility is reset after changing group or adding filter for aggregated metrics #42540

Open
maxzheng opened this issue May 10, 2024 · 1 comment
Labels
.Frontend Priority:P2 Average run of the mill bug Querying/GUI Query builder catch-all, including simple mode .Team/QueryingComponents Type:Bug Product defects

Comments

@maxzheng
Copy link
Contributor

Describe the bug

I have a question that groups by Industry with a 5 metrics where only 2 of them are displayed in a stacked bar chart.
When I change the group to another field in the editor, all 5 metrics are displayed instead of just the 2.

If you add a filter for one of the metrics (count) to be greater than 10, then it will remove all the metrics and nothing is shown in the chart and you are forced to re-select the metrics.

To Reproduce

  1. Create question with 5 metrics grouped by one field and show only 2 of the metrics in a stacked bar chart
  2. Change grouping to another field
  3. Observe all 5 metrics are shown
  4. Open the editor again and add a filter for one of the metrics (like count > 10) and all metrics are removed. You will need to select the metrics in the viz settings.

Expected behavior

Changing group field or adding filter for aggregated metrics should not change visualization behavior. Doing so makes Metabase feel like a beta quality product.

Logs

n/a

Information about your Metabase installation

Master on Postgres

Severity

There is a workaround but it makes Metabase look bad / poor quality, so sooner we fix the better to improve user experience & impression

Additional context

No response

@maxzheng maxzheng added Type:Bug Product defects Priority:P2 Average run of the mill bug .Frontend labels May 10, 2024
@ranquild ranquild added Querying/GUI Query builder catch-all, including simple mode .Team/QueryingComponents labels May 20, 2024
@ranquild
Copy link
Contributor

Adding QC because this is cased by field references used in the viz settings; should be fixed by migrating them to aliases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
.Frontend Priority:P2 Average run of the mill bug Querying/GUI Query builder catch-all, including simple mode .Team/QueryingComponents Type:Bug Product defects
Projects
None yet
Development

No branches or pull requests

2 participants