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

[chart]Time Series Bar Chart does not sort #13792

Closed
lctdulac opened this issue Mar 25, 2021 · 11 comments · Fixed by #15343
Closed

[chart]Time Series Bar Chart does not sort #13792

lctdulac opened this issue Mar 25, 2021 · 11 comments · Fixed by #15343
Assignees
Labels
!deprecated-label:bug Deprecated label - Use #bug instead viz:charts:bar Related to the Bar chart

Comments

@lctdulac
Copy link

Hi SuperSet community,
I encountered a bug where my Time Series Bar Chart is not sorting correctly.

Expected results

Sort in this order : Trusted > Neutral > Low > Medium > High

Screenshot 2021-03-25 at 11 46 08

Actual results

Sorted like : Low > Medium > High > Trusted > Neutral

Screenshot 2021-03-25 at 11 46 13

Environment

Superset Version: 1.0.1

Additional context

Other charts seem to be sorted correctly with same logic (Table chart) :

image

image

Thanks for your hard work.

@lctdulac lctdulac added the #bug Bug report label Mar 25, 2021
@junlincc junlincc added viz:charts:bar Related to the Bar chart enhancement:request Enhancement request submitted by anyone from the community and removed #bug Bug report labels Mar 26, 2021
@junlincc junlincc changed the title Time Series Bar Chart does not sort [chart]Time Series Bar Chart does not sort Mar 26, 2021
@junlincc junlincc added !deprecated-label:bug Deprecated label - Use #bug instead and removed enhancement:request Enhancement request submitted by anyone from the community labels Mar 27, 2021
@junlincc
Copy link
Member

same as #13807

@john-bodley
Copy link
Member

Related to #9998.

@rusackas
Copy link
Member

rusackas commented Jul 6, 2021

@zhaoyongjie any advice on how to move this forward?

@zhaoyongjie zhaoyongjie self-assigned this Jul 8, 2021
@zhaoyongjie
Copy link
Member

@zhaoyongjie any advice on how to move this forward?

let me dive in this issue.

@junlincc junlincc removed the P0 Priority item - Critical label Jul 19, 2021
@fdzuJ
Copy link

fdzuJ commented May 2, 2023

On 2.1.0 in time-series bar chart dimensions are still sorted alphabetically (irrespective of sort by clause)

@zhaoyongjie
Copy link
Member

zhaoyongjie commented May 2, 2023 via email

@fdzuJ
Copy link

fdzuJ commented May 2, 2023

the time-series bar chart is a legacy chart, please check the echart version for sorting.

There are two time-series bar charts - one marked as legacy, the other is not. I'm using the one which is not marked. Also it has "ECharts" tag.
bar charts
echart

@zhaoyongjie
Copy link
Member

zhaoyongjie commented May 2, 2023 via email

@fdzuJ
Copy link

fdzuJ commented May 2, 2023

Okay, makes sense, could you explain what is "dimensions are still sorted alphabetically" and what you want?

Exactly the same as original issue.
I've got metric in 4 categories (in example they are 17, 18, 19 and 20). But in stacked bar chart I want them to appear in order 19, 20, 17, 18.
Here is "SORT BY" clause I use:
sort by
And here is resulting chart still sorted alphabetically:
изображение

@zhaoyongjie
Copy link
Member

zhaoyongjie commented May 2, 2023 via email

@fdzuJ
Copy link

fdzuJ commented May 3, 2023

I enabled GENERIC_CHART_AXES feature flag which changed chart names and how I pick x axis, but didn't change ordering with same "SORT BY" - where else should I set how to sort metric or dimension? I can't find any other relative option.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
!deprecated-label:bug Deprecated label - Use #bug instead viz:charts:bar Related to the Bar chart
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants