"Could not determine delimiter" when trying to render TSV via stdin about rich-cli HOT 5 OPEN

hi,
and/or adding a --delim option (or something similar) on the command line to force the definition (in case of detection problem for example)...
regards.

from rich-cli.

OK, apparently the problem is with only sniffing truncated data ([:1024]) which can break the CSV sniffer algorithm as it tries to detect the delimiter by counting the occurrences on each line (and truncating in the middle of a line will therefore corrupt the data for the sniffer).

Changing the logic to sniff the first N lines instead of first 1024 characters would solve this issue.

from rich-cli.

OK, apparently the problem is with only sniffing truncated data ([:1024]) which can break the CSV sniffer algorithm as it tries to detect the delimiter by counting the occurrences on each line (and truncating in the middle of a line will therefore corrupt the data for the sniffer).

Wow, is that the reason why?!? 😲 I've been wondering for years why the code example in the official Python csv.Sniffer docs does not seem to work. I never realized it is because it breaks in the middle of a line. 🤨

Seems to me this should be fixed in the official Python docs as well, since I've never managed to get it to work...

Anyway, thanks for this gem! 😊

from rich-cli.

Is there any --delim or similar option to force delim detection?

from rich-cli.

I just submitted a pull request to add --csv-format that lets you set the dialect to use.

from rich-cli.