Skip to content

Added benchmark for LinqToCsv#21

Merged
joelverhagen merged 4 commits intojoelverhagen:mainfrom
JoshClose:linq-to-csv
Jan 28, 2021
Merged

Added benchmark for LinqToCsv#21
joelverhagen merged 4 commits intojoelverhagen:mainfrom
JoshClose:linq-to-csv

Conversation

@JoshClose
Copy link
Contributor

Code is based off of section "Reading Raw Data Rows" https://www.codeproject.com/Articles/25133/LINQ-to-CSV-library#ReadingRawDataRows

@JoshClose JoshClose mentioned this pull request Jan 23, 2021
@joelverhagen
Copy link
Owner

Looks like this one is also outputting data different:

 Group #1 (result JSON length = 921):
   - Csv
   - CsvHelper
   - CsvTextFieldParser
   - Ctl_Data
   - Cursively
   - FastCsvParser
   - FluentCsv
   - HomeGrown
   - LumenWorksCsvReader
   - mgholam_fastCSV
   - Microsoft_VisualBasic_FileIO_TextFieldParser
   - NReco_Csv
   - ServiceStack_Text
   - string_Split
   - SoftCircuits_CsvParser
   - Sylvan_Data_Csv
   - TinyCsvReader
 
 Group #2 (result JSON length = 939):
   - LinqToCsv

@JoshClose
Copy link
Contributor Author

Is there a way to test this in your source? I'll make sure they are working with that check before making a pull request.

@joelverhagen
Copy link
Owner

My approach is to use the Test Explorer in Visual Studio. This will pop up the tests in the solution and you can run them there.

@JoshClose
Copy link
Contributor Author

Oh, of course. lol Not sure what I was thinking.

@jzabroski
Copy link

Check encoding.

@JoshClose
Copy link
Contributor Author

This library is returning null for empty strings.

@JoshClose
Copy link
Contributor Author

Fixed.

@joelverhagen joelverhagen merged commit 9017764 into joelverhagen:main Jan 28, 2021
@joelverhagen
Copy link
Owner

Awesome!

@JoshClose JoshClose deleted the linq-to-csv branch January 28, 2021 18:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants