peterjc / thapbi-pict

Tree Health and Plant Biosecurity Initiative - Phytophthora ITS1 Classifier Tool
https://thapbi-pict.readthedocs.io/
MIT License
8 stars 2 forks source link

Use thousands separator for read counts in the Excel reports #599

Closed peterjc closed 4 months ago

peterjc commented 4 months ago

I've found myself having to do this manually a lot recently for interpretting analyses - might as well do it by default.

(Not sure of Excell lets you do this in a localisation respecting way though, not everyone uses commas as a 1000s separator).

peterjc commented 4 months ago

https://xlsxwriter.readthedocs.io/format.html#number-formats-in-different-locales says:

Excel handles this by storing the number format in the file format in the US locale, in this case #,##0.00, but renders it according to the regional settings of the host OS.

i.e. We can use #,###,##0 or similar and it should respect regional settings.