[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Textscan and csv fitness data problem

From: Ben Abbott
Subject: Re: Textscan and csv fitness data problem
Date: Wed, 03 Jan 2018 12:14:35 -0800

> On Jan 3, 2018, at 12:19 AM, PhilipNienhuis <address@hidden> wrote:
> bpabbott wrote
>>> On Jan 1, 2018, at 11:51 AM, PhilipNienhuis &lt;
>> pr.nienhuis@
>> &gt; wrote:
>>> NJank wrote
>>>> On Jan 1, 2018 12:13 PM, "PhilipNienhuis" &lt;
>>>> pr.nienhuis@
>>>> &gt; wrote:
>>>> NJank wrote
>>>>> On Jan 1, 2018 9:06 AM, "Thomas Esbensen"
>>>> As to textscan, Dan did a lot of good work lately, I think the bugs you
>>>> implied have been fixed in the development branch.
>>>> Yeah, i noticed that. Would those make it into a 4.2.2 or not until
>>>> 4.4.0?
>>>> Been keeping my fingers crossed that it would suddenly "just work" and I
>>>> wouldn't have to dive into his data again.
>>> Have a look in the log:
>>> bugs 52116 and 52479 have been fixed on stable, the last one (bug 52550)
>>> not. If you want you can ask in the latter bug report to backport it to
>>> stable.
>>> As to csv2cell's erroneous column conversion, I've fixed that stupid bug
>>> and
>>> pushed it. To use it, get from here:
>>> and then do
>>> mkoctfile
>>> to build a fixed version. Swap it into place, using 
>>> "pkg load io; which csv2cell" 
>>> to find out where it should live, followed by 
>>> "pkg unload io; clear -f"
>>> to clear the way for copying (otherwise csv2cell.oct is locked), and then
>>> copy csv2cell.oct into place.
>>> Philip
>> The original file has lines with a varied number of columns.  As a result
>> …
>> error: csv2cell: incorrect CSV file, line 2 too short
>> The first row are the column labels (127 of them), and 2nd row only has 19
>> columns (18 commas). There are other rows deep in the file with 127
>> columns too.
> Sure but if you try csv2cell with as 2nd argument a spreadsheet-style range,
> it'll read .csv files with a varying nr. of data per row just fine, see my
> first answer in this thread.
> If you want to read all of the file, just supply a sufficiently large range;
> it'll fill empty fields beyond current line length with "" (empty string).
> See "help csv2cell"
> The only practical limit is the max line length of 4096 chars (a #DEFINEd
> setting; changing that is easy as csv2cell() is just an .oct file).
> (of course, as usual I can only vouch for csv2cell() to work fine on the 4
> boxes I have access to: my 2 multiboot Linux/Win7/Win10 boxes + 2 Win7 boxes
> at work.)
> Philip

Ok. I’m not familiar with the history behind the default behavior. I was 
expecting the default behavior to load the full csv file. Given the current 
design, I don’t know how to determine the actual size of the file. Meaning when 
a range of rows/cols is specified, there is now way to be sure all the 
information is included.

If the behavior were changed such that "error: csv2cell: incorrect CSV file, 
line 2 too short” was replaced by “warning: csv2cell: line 2 has fewer columns 
than the prior lines” and the entire file were to be read would there be an 
adverse impact on compatibility?


reply via email to

[Prev in Thread] Current Thread [Next in Thread]