search

OpenWaterFoundation / cdss-app-snodas-tools

Colorado's Decision Support Systems (CDSS) Snow Data Assimilation System (SNODAS) Tools
8 stars 4 forks source link

CSV file header has CR #1

Closed smalers closed 7 years ago

smalers commented 7 years ago

The ByDate CSV files have a CR character at the end of the header and all others have CRLF. The end of lines need to be consistent so they don't confuse software like editors. See if this is an easy thing to fix. Note that in code \r is carriage return and \n is newline.

Attaches is a screen shot from Notepad++ that illustrates this. Use the View / Show Symbol / Show End of Line menu in Notepad++. See the attached screen shot illustrating. csv-header-issue

Files generated on Windows should usually have CRLF. Files generated on Linux should usually have LF. Therefore it is odd to only see CR.

egiles16 commented 7 years ago

Done. The SNODAS_utilities Python script no longer produces the extra row between the header row and the data rows (updated in the GitHub repository). Also, I updated all of the historical SnowpackStatisitcsByDate CSV files so that the extra row is deleted.

smalers commented 7 years ago

OK. Accept as completed. Closing.