SolarArbiter / solarforecastarbiter-core

Core data gathering, validation, processing, and reporting package for the Solar Forecast Arbiter
https://solarforecastarbiter-core.readthedocs.io
MIT License
33 stars 21 forks source link

Update Download metrics script to quote string fields. #776

Open lboeman opened 2 years ago

lboeman commented 2 years ago

The download metrics function does not handle string fields containing commas correctly. To create a properly formatted csv, we should quote any string fields such as "name" so they don't create new columns with any included commas. https://github.com/SolarArbiter/solarforecastarbiter-core/blob/881fbab44165370ea1adbdf53149302ca26c0af9/solarforecastarbiter/reports/templates/html/macros.j2#L311