Resolve "SampleSelection strategy "byclass", csv parser not working"

Bug with CR-LF line endings in SamplingRateCalculator.

Closes #1549 (closed)

Hi, The code for this parser is very complex from my point of view. I have a simpler version here: jinglada/otb@fddfca31 I does not deal with the bug you solved, but it could be worth using a simpler implementation. Also, I was thinking that we could have a general purpose CSV parser that could be used elsewhere in OTB.

+1 for a general purpose CSV parser.

In the current implementation, my intent was not to copy the strings, but use iterators instead (for performance issues). I don't mind trying to simplify the implementation, but it may not be possible if we go anyway for a generic parser.

Would something like boost::string_ref help avoid copies?

added 75 commits

• e8f2558f...01a00be6 - 74 commits from branch develop
• 02ed52cc - MRG: Synchronize with branch develop

Compare with previous version

merged

mentioned in commit 1eb35e74

added bug label