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

wrong sort order in time series #1061

Closed
Rdataflow opened this issue Jun 7, 2023 · 2 comments · Fixed by #1062
Closed

wrong sort order in time series #1061

Rdataflow opened this issue Jun 7, 2023 · 2 comments · Fixed by #1062
Assignees
Labels
bug Something isn't working

Comments

@Rdataflow
Copy link
Contributor

Describe the bug
There is a wrong sort order in some time series applied.

To Reproduce

  1. https://test.visualize.admin.ch/v/kSH3lpmO-vS2?dataSource=Test
  2. See sort order in time series (1988, 2021, 1989, ...)

Expected behavior
Ascending sort order

Screenshots or video
image

Environment (please complete the following information):

  • Visualize environment and version: TEST v3.19.19 (7dc9228)
  • Browser and version: Edge 113.0.1774.50

@bprusinowski @sosiology rapid fix needed for ongoing deploy ! THX
cc @ZellerSabine @ortnever

@Rdataflow Rdataflow added the bug Something isn't working label Jun 7, 2023
@ortnever
Copy link

ortnever commented Jun 7, 2023

thank you thomas for reporting the bug. I'm not sure if it's related, but I've also noticed this problem when sorting tables.
to reproduce :

  • open the dataset "Emission de gaz à effet de serre par secteur et par gaz" on INT : https://int.visualize.admin.ch/fr/create/0FjGoLHrtadT?dataSource=Int
  • Select the Table chart type
  • Sort Dimension "Year" using the arrow OR via the left-hand menu by selecting the "Year" dimension.
  • In addition to the fact that 2021 is displayed between 2020 and 2019, I've noticed that the ascending sort doesn't work at all.

tested with v3.19.19 ([7dc9228]

see screenshot.
image

image

@ortnever
Copy link

Checked by OV. Ok. Thank you !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants