search

phoronix-test-suite / test-profiles

A read-only Git copy of the OpenBenchmarking.org test profiles.
Other
68 stars 80 forks source link

Commas in test names break csv files #294

Open lucic71 opened 10 months ago

lucic71 commented 10 months ago

https://github.com/phoronix-test-suite/test-profiles/blob/bde43b6155829b40b804cf9dc1ea3bb334f5bf29/pts/compress-zstd-1.6.0/test-definition.xml#L38

Parsing csv files that have test names which contain commas is really ugly. We should delete these commas.

lucic71 commented 3 months ago

The same issue is present in sqlite-speedtest-1.0.1: SQLite Speedtest - Timed Time - Size 1,000 (sec). Notice the comma in the test name.