frictionless-js icon indicating copy to clipboard operation
frictionless-js copied to clipboard

Sniffing CSV dialect: an option not the default?

Open rufuspollock opened this issue 5 years ago • 3 comments

Atm we auto-sniff the CSV dialect if it is not provided. This adds complexity (and overhead) to use.

Maybe we could switch to being an option rather than the default.

@anuveyatsu any idea why we started doing this?

rufuspollock avatar Jun 12 '19 19:06 rufuspollock

@rufuspollock we started doing this to support different field delimiters such as semicolon etc.

anuveyatsu avatar Jun 19 '19 17:06 anuveyatsu

@anuveyatsu makes sense. Do you think it should be optional or not?

rufuspollock avatar Jun 20 '19 08:06 rufuspollock

@rufuspollock I might be wrong but as I know in some countries, e.g., France and Italy ; is used more frequently than , as a field delimiter. I've also seen some sources with tab delimited fields. However, I'm not sure how often they are. I think we should keep it as default right now and wait for an issue from users.

anuveyatsu avatar Jun 21 '19 04:06 anuveyatsu