[issue29405] improve csv.Sniffer().sniff() behavior

2017-02-03 Thread Milt Epstein
Milt Epstein added the comment: That's right, with 11 lines in the sample data, total will become 20 on the second iteration. And that throws off some of the computations done in that function. Your patch looks good, in that it will achieve what I'm requesting. But :-), your pointing out

[issue29405] improve csv.Sniffer().sniff() behavior

2017-01-31 Thread Milt Epstein
Milt Epstein added the comment: FWIW, it might be more concise and more consistent with the existing code to change the one line to: total = min(float(chunkLength * iteration), float(len(data))) -- ___ Python tracker <

[issue29405] improve csv.Sniffer().sniff() behavior

2017-01-31 Thread Milt Epstein
New submission from Milt Epstein: I'm trying to use csv.Sniffer().sniff(sample_data) to determine the delimiter on a number of input files. Through some trial and error, many "Could not determine delimiter" errors, and analyzing how this routine works/behaves, I settled on sample_