medea-project / ipcc-fact-checking

GOAL: to establish a line of sight between data and its source
Creative Commons Attribution 4.0 International
1 stars 2 forks source link

Include line feed character (LF) at end of lines #3

Closed eric-brechemier closed 10 years ago

eric-brechemier commented 10 years ago

Depending on operating system and configuration, the end of lines in a text file can be marked by different characters: CR (carriage return) and/or LF (line feed).

The 1994 Appendix 3 data uses only CR characters as line separators (no LF).

Git considers only LF as a line separator, and thus considers that the file has a single line.

This makes it hard to review changes, because the whole file appears different when only 1 line has been modified:

https://github.com/medea-project/ipcc-fact-checking/commit/c1ebc30b0d1d5bb1645e367a23249060cf407eba

@apoloniarakow @monlafon Like for the field separator in issue #2, this may be an option to select when saving the file to CSV. The end of line character 'line feed' may be referred as LF or \n, while 'carriage return' is referred as CR or '\r'.

Let me know if you find something related.

eric-brechemier commented 10 years ago

Here again, the settings are correct in Annex H of 1997 Special Report, which uses line feed (LF) as line separators.

eric-brechemier commented 10 years ago

Same issue in 2000 SR Emissions:

https://github.com/medea-project/ipcc-fact-checking/blob/master/ipcc.ch/2000-ipcc-sr-emissions/annexII/data.csv

monlafon commented 10 years ago

Let me know if this problem was fixed by using OpenOffice

eric-brechemier commented 10 years ago

Yes, the expected LF character is now used as end of line character.

I close the issue.