Re: bug: orgtbl-to-tsv: premature end of table

[Andreas Leha <andreas.leha@med.uni-goettingen.de>]

Hi all,

"Charles C. Berry" <ccberry@ucsd.edu> writes:
> On Wed, 22 Jul 2015, Nicolas Goaziou wrote:
>
>> Hello,
>>
>> "Charles C. Berry" <ccberry@ucsd.edu> writes:
>>
>>> Not a bug IMO.
>>
>> Notwithstanding the R issue, I think trimming the output of a radio
>> table is a bit aggressive. We cannot know if trailing (and leading)
>> spaces are significant in the output format the user defined.
>>
>> I removed the trimming part in commit
>> bf37cd09b128b4431e1dd1a538fbcc4be3615042.
>>
>> Let me know if you disagree with that change.
>
> I don't think the change broke anything that wasn't already broken.
>
> And my diagnosis was wrong - I was confused by a problem that remains:
>
> `org-babel-R-assign-elisp' is broken. Ragged rows in a table are filled 
> with "" in each missing field; `org-table-to-lisp' returns a list of lists 
> of common length.
>
> `org-babel-R-assign-elisp' inspects that list to see if any element vary 
> in length. But they do not, and therefore `max' and `min' in the let* 
> varlist are equal and later on `ob-R-transfer-variable-table-with-header' 
> is always used.
>
> I think lom pik's example would not have failed had max and min differed 
> as `ob-R-transfer-variable-table-without-header' would have been used 
> instead.
>
> R src blocks ought to render missing elements due to ragged rows in a 
> table as NA, but this isnt the current behavior for this table:
>
> #+NAME: tbl-ragged
> | 1 | 2 | a |
>
> | 8 | 9 |   |
>
> It seems like the fix for this would be to prune any "" at the ends of 
> lists in the value arg when beginning  `org-babel-R-assign-elisp'.
>
> But I am tired, so I'll defer trying this till another time.
>
> Best,
>
> Chuck

Thank you all for looking into this.  I have seen this as well and the
manual work-around (to put sth into the last cell) was getting really
annoying -- especially if the table was auto-generated by sth else.

While looking at handling tables, could I suggest to switch to csv
instead of tsv for the table passing?  I still see issues with tables
not separated.  (This happens only in preview (C-c C-v v) and not when
executing the block.)  I never had time to track this down, but I get
caught now and then.  Using csv should be more robust in this regard.

Thanks,
Andreas