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

Number format and data sorting (e.g. years) in the xls download #1142

Closed
sosiology opened this issue Aug 22, 2023 · 4 comments · Fixed by #1146
Closed

Number format and data sorting (e.g. years) in the xls download #1142

sosiology opened this issue Aug 22, 2023 · 4 comments · Fixed by #1146
Assignees

Comments

@sosiology
Copy link
Contributor

Currently the data downloaded in xls

  • are not sorted as in the graph (e.g. the years are not sorted chronologically)
  • have a "standard" format. The user has to reformat them if he wants to make a graph in excel.

When downloading, the order of data and Data Format must be kept. In particular, numbers must be exported as number.

Image

@sosiology
Copy link
Contributor Author

Tested with this chart: https://test.visualize.admin.ch/v/eaJr-s6bZOv6?dataSource=Int

  • CSV sorted by date
  • Excel Sorted by date
  • In Excel the values appear with a green notification indicating the number is stored as text
    • This issue can be solved by double clicking in the cell or selecting all values and using the excel prompt to convert values to numbers

Image

Image

@bprusinowski
Copy link
Collaborator

Hi @sosiology, thanks for testing!

The current approach is to keep dates as strings, even when their resolution equals Year. We can change this to convert dates to strings only when they're not yearly dates – let me know what you think about this.

For the Hierarchie dimension, we can't really do a lot, as it's marked as NominalDimension, without a numerical data type (it's simply marked as string dimension in Cube Creator) – so it appears as number in the export, but technically shouldn't be treated like one, at least according to the metadata 👀

@sosiology
Copy link
Contributor Author

@bprusinowski tested by BAFU and approved 🎉

@bprusinowski
Copy link
Collaborator

Nice! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants