go-pluto / styx

Export Data from Prometheus to csv, gnuplot & matplotlib
https://speakerdeck.com/metalmatze/prometheus-styx
MIT License
99 stars 24 forks source link

Seperators #14

Closed Knuspel closed 3 years ago

Knuspel commented 3 years ago

Changed seperators to semicolons for Excel compatibility

metalmatze commented 3 years ago

Thanks! I'm really quite surprised that importing with Excel should work with , and only with ;. Have you tried this? https://superuser.com/a/407085

Do you think it's still easier with a semicolon?

Knuspel commented 3 years ago

Generally I'd agree with the mentioned solution. However it collides with the labels from Prometheus because they are also separated with ','. So unless I've missed something the different separator makes using the files easier.

If separator strings are important, maybe it could be added as a optional input parameter?

metalmatze commented 3 years ago

However it collides with the labels from Prometheus because they are also separated with ','.

No that makes 100% sense now. I didn't see that one before! :+1:

Thanks for taking the time!