ITxPT / DATA4PTTools

Shared space for the development of the DATA4PT Greenlight NeTEx validation tool(s)
MIT License
10 stars 2 forks source link

"error_count" does not match "errors" #6

Closed skinkie closed 2 years ago

skinkie commented 2 years ago

In my ideal world the human readable errors would be unique, but contain a list with references to the exact positions in the file, for simplicity reasons: line based. That will obviously not always work, but I think that can be a good input criteria.

I currently see an error count in thousands, but I only see messages in the thirties. That does not make sense.

ollelar commented 2 years ago

This is currently by design for performance reasons. We stop after 32 errors.

skinkie commented 2 years ago

This is currently by design for performance reasons. We stop after 32 errors.

What exactly do you stop with. The writing to a file or finding more errors? Because it doesn't makes sense that you have an 'error_count' that is much higher.

lekotros commented 2 years ago

Fixed in #15