NCsvPerf

NCsvPerf copied to clipboard

Make the benchmark unit test pass in the presence of quoted fields.

2

The NCsvPerf benchmark PackageAssets.csv file doesn't contain quotes, but all existing parsers can be configured to produce consistent results in the presence of a quoted field. This PR adds the...

MarkPflug

Usually we should _program to interfaces, not implementations_, to abstract implementation details and focus on behaviour. I think it is more fair does not obligate implementations to return a specific...

leandromoh

Add tests for write performance

1

I do understand the goal of this project and the tests being done, but I think a nice addition (that makes sense in this context) would be to have a...

pjfsilva

Add quoted values to the test CSV and possibly some larger blocks.

13

Just a suggestion, but having a column with quoted and quote escapes will round out the tests. I know it might break someone's code, but I realized when diving in...

electricessence

Data Mapping benchmark

2

See #5 discussion - ML.NET doesn't have low-level API, so some Csv readers need a higher-level benchmark in order to participate.

jzabroski

add new parser based on Utf8StreamReader for comparison

3

hello, can you in your benchmarks add new parser based on [Utf8StreamReader](https://github.com/Cysharp/Utf8StreamReader) ?

DrJakiellSWO