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

Pivot table summary returning wrong results #16952

Closed
djouallah opened this issue Oct 4, 2021 · 14 comments
Closed

Pivot table summary returning wrong results #16952

djouallah opened this issue Oct 4, 2021 · 14 comments
Assignees
Labels
#bug Bug report

Comments

@djouallah
Copy link

djouallah commented Oct 4, 2021

I am using Preset Cloud, so not sure which version is used, as far as I can see Pivot table summary generate only 1 SQL Query, I would expect that it generate two.

  • one for the rows
  • another for the summary

see the results of the division which is wrong Actual/ Budget, I don't know how to tell superset to keep the same metric definition for the summary rows
image

@djouallah djouallah added the #bug Bug report label Oct 4, 2021
@grafke
Copy link
Contributor

grafke commented Nov 16, 2021

+1

@rusackas
Copy link
Member

I'm not so sure this is a common use case. The Total (Sum) row is just that, a sum of the values in the column. By that definition, this chart is quite correct. If you want to use these totals formulaically against each other, that seems like it would be adding spreadsheet functionality that goes beyond the typical scope of a pivot table. @kgabryje @villebro curious your thoughts on this, if they differ.

@djouallah
Copy link
Author

yes, it is the default expectation how the measure works in any BI tools, I expect the summary row to respect the measure definition not doing a sum, if I want a sum, then I create a measure that explicitly say, for a summary row do a sum of a column

@robjuffermans
Copy link

robjuffermans commented Nov 22, 2021

@djouallah Correct me if I'm wrong but the problem is the summing of the percentages in the Progress column in the Total (sum) row, right?
This should be a weighted percentage. Summing percentages is useless.

@djouallah
Copy link
Author

@robjuffermans what I am saying having an option to do a sum in the first place is a wrong, I expect the summary row to respect the metric definition , that pretty much how any BI tools works (Tableau, PowerBI, Google Data Studio etc)

@djouallah
Copy link
Author

something like this see the percentage works at all level even in the summary rows
image

@robjuffermans
Copy link

exactly, that's what I'm missing also. @rusackas this should be regarded normal functionality for a pivot table from my point of view.

@stale
Copy link

stale bot commented Apr 17, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. For admin, please label this issue .pinned to prevent stale bot from closing the issue.

@stale stale bot added the inactive Inactive for >= 30 days label Apr 17, 2022
@stale stale bot removed the inactive Inactive for >= 30 days label Jan 18, 2023
@rusackas
Copy link
Member

CC @lauderbaugh and @yousoph in case you know how we can revisit this (and related issues) holistically. Adding @kgabryje since he's a SME on the implementation

@mtthsbrr
Copy link

@djouallah Did you find any workarounds already. I could imagine to build a chart per aggregation level but it feels far from perfect.

Ideally one can fold/unfold on all the dimension and (sub)totals should always respect the formula behind the metric instead of creating a sum of the metric results.

@djouallah
Copy link
Author

@mtthsbrr nope, just gave up

@wkolbnik
Copy link

+1

1 similar comment
@jjlabuda
Copy link

jjlabuda commented Apr 4, 2024

+1

@rusackas
Copy link
Member

rusackas commented Jun 3, 2024

There are quite a few issues around this, which I'm trying to consolidate. I'll close this issue in favor of another that sums it up fairly concisely: #25747

@rusackas rusackas closed this as completed Jun 3, 2024
@rusackas rusackas reopened this Jun 3, 2024
@rusackas rusackas closed this as not planned Won't fix, can't repro, duplicate, stale Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
#bug Bug report
Projects
None yet
Development

No branches or pull requests

9 participants