emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
* [babel] [BUG] table with header line as parameter
@ 2012-05-09 22:14 Andreas Leha
  2012-05-11 14:57 ` Andreas Leha
  0 siblings, 1 reply; 3+ messages in thread
From: Andreas Leha @ 2012-05-09 22:14 UTC (permalink / raw)
  To: emacs-orgmode

Hi all,

I do not know what could be the cause of this, but I can't have a
table with header as argument to a source block any more:

#+name: table_w_header
| one | two |
|-----+-----|
| 1   | 3   |

#+begin_src R :var tbl=table_w_header

#+end_src

If I do C-c C-v v on the above source block, I get this backtrace:

,----
| Debugger entered--Lisp error: (wrong-type-argument sequencep hline)
|   length(hline)
|   mapcar(length (("one" "two") hline (1 3)))
|   org-babel-R-assign-elisp(tbl (("one" "two") hline (1 3)) nil nil)
|   #[(pair) "\302\b@\bA\303\304	\"A\305\232\303\306	\"A\305\232$\207" [pair params org-babel-R-assign-elisp assoc :colnames "yes" :rownames] 7]((tbl ("one" "two") hline (1 3)))
|   mapcar(#[(pair) "\302\b@\bA\303\304	\"A\305\232\303\306	\"A\305\232$\207" [pair params org-babel-R-assign-elisp assoc :colnames "yes" :rownames] 7] ((tbl ("one" "two") hline (1 3))))
|   org-babel-variable-assignments:R(((:cache . "no") (:colname-names (tbl "one" "two")) (:comments . "") (:exports . "code") (:hlines . "no") (:noweb . "no") (:padline . "") (:padnewline . "yes") (:result-params "replace") (:result-type . value) (:results . "replace") (:rowname-names) (:session . "none") (:shebang . "") (:tangle . "no") (:var tbl (1 3))))
|   org-babel-expand-body:R("\n" ((:cache . "no") (:colname-names (tbl "one" "two")) (:comments . "") (:exports . "code") (:hlines . "no") (:noweb . "no") (:padline . "") (:padnewline . "yes") (:result-params "replace") (:result-type . value) (:results . "replace") (:rowname-names) (:session . "none") (:shebang . "") (:tangle . "no") (:var tbl (1 3))))
|   org-babel-expand-src-block()
|   call-interactively(org-babel-expand-src-block nil nil)
`----

I tested on with emacs -Q on
1. GNU Emacs 24.1.50.1 (x86_64-pc-linux-gnu, GTK+ Version 3.4.1) of 2012-05-02 on zelenka, modified by Debian
2. GNU Emacs 23.4.1 (x86_64-pc-linux-gnu, GTK+ Version 2.24.10) of 2012-04-07 on trouble, modified by Debian

My org-version: Org-mode version 7.8.09 with last commit
,----
| commit 56520a6936351d6fcc149652e1ff30f50c43eb4e
| Merge: 505cc7a 0ebeb77
| Author: Nicolas Goaziou <n.goaziou@gmail.com>
| Date:   Wed May 9 23:34:05 2012 +0200
| 
|     Merge branch 'maint'
`----
So, the fix discussed in
[[https://lists.gnu.org/archive/html/emacs-orgmode/2011-02/msg01444.html]]
should be there.


Not sure if this could be connected:  I updated my emacs-snapshot (the
24 version) from [[http://emacs.naquadah.org]] today.


Any help is highly appreciated,
Andreas

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [babel] [BUG] table with header line as parameter
  2012-05-09 22:14 [babel] [BUG] table with header line as parameter Andreas Leha
@ 2012-05-11 14:57 ` Andreas Leha
  2012-05-11 20:51   ` Eric Schulte
  0 siblings, 1 reply; 3+ messages in thread
From: Andreas Leha @ 2012-05-11 14:57 UTC (permalink / raw)
  To: emacs-orgmode

Andreas Leha <andreas.leha@med.uni-goettingen.de> writes:

> Hi all,
>
> I do not know what could be the cause of this, but I can't have a
> table with header as argument to a source block any more:
>
> #+name: table_w_header
> | one | two |
> |-----+-----|
> | 1   | 3   |
>
> #+begin_src R :var tbl=table_w_header
>
> #+end_src
>
> If I do C-c C-v v on the above source block, I get this backtrace:
>
> ,----
> | Debugger entered--Lisp error: (wrong-type-argument sequencep hline)
> |   length(hline)
> |   mapcar(length (("one" "two") hline (1 3)))
> |   org-babel-R-assign-elisp(tbl (("one" "two") hline (1 3)) nil nil)
> |   #[(pair) "\302\b@\bA\303\304 \"A\305\232\303\306 \"A\305\232$\207"
> | [pair params org-babel-R-assign-elisp assoc :colnames "yes"
> | :rownames] 7]((tbl ("one" "two") hline (1 3)))
> |   mapcar(#[(pair) "\302\b@\bA\303\304 \"A\305\232\303\306
> | \"A\305\232$\207" [pair params org-babel-R-assign-elisp assoc
> | :colnames "yes" :rownames] 7] ((tbl ("one" "two") hline (1 3))))
> |   org-babel-variable-assignments:R(((:cache . "no") (:colname-names
> | (tbl "one" "two")) (:comments . "") (:exports . "code") (:hlines
> | . "no") (:noweb . "no") (:padline . "") (:padnewline . "yes")
> | (:result-params "replace") (:result-type . value) (:results
> | . "replace") (:rowname-names) (:session . "none") (:shebang . "")
> | (:tangle . "no") (:var tbl (1 3))))
> |   org-babel-expand-body:R("\n" ((:cache . "no") (:colname-names (tbl
> | "one" "two")) (:comments . "") (:exports . "code") (:hlines . "no")
> | (:noweb . "no") (:padline . "") (:padnewline . "yes")
> | (:result-params "replace") (:result-type . value) (:results
> | . "replace") (:rowname-names) (:session . "none") (:shebang . "")
> | (:tangle . "no") (:var tbl (1 3))))
> |   org-babel-expand-src-block()
> |   call-interactively(org-babel-expand-src-block nil nil)
> `----
>
> I tested on with emacs -Q on
> 1. GNU Emacs 24.1.50.1 (x86_64-pc-linux-gnu, GTK+ Version 3.4.1) of 2012-05-02 on zelenka, modified by Debian
> 2. GNU Emacs 23.4.1 (x86_64-pc-linux-gnu, GTK+ Version 2.24.10) of 2012-04-07 on trouble, modified by Debian
>
> My org-version: Org-mode version 7.8.09 with last commit
> ,----
> | commit 56520a6936351d6fcc149652e1ff30f50c43eb4e
> | Merge: 505cc7a 0ebeb77
> | Author: Nicolas Goaziou <n.goaziou@gmail.com>
> | Date:   Wed May 9 23:34:05 2012 +0200
> | 
> |     Merge branch 'maint'
> `----
> So, the fix discussed in
> [[https://lists.gnu.org/archive/html/emacs-orgmode/2011-02/msg01444.html]]
> should be there.
>
>
> Not sure if this could be connected:  I updated my emacs-snapshot (the
> 24 version) from [[http://emacs.naquadah.org]] today.
>
>
> Any help is highly appreciated,
> Andreas


Hi all,

sorry for answering to my own message.

I thought, that I did not pull between working state and broken state, but
apparently I did.

So, in the end, =git bisect= now told me the responsible commit:
,----
| fb7ebd2dae66a7b42aecff695fe40461a33a76ed is the first bad commit
| commit fb7ebd2dae66a7b42aecff695fe40461a33a76ed
| Author: Eric Schulte <eric.schulte@gmx.com>
| Date:   Wed Apr 25 15:34:15 2012 -0400
| 
|     clean up the code implementing reads of irregular data into R
|     
|     * lisp/ob-R.el (org-babel-R-assign-elisp): Clean up the code
|       implementing reads of irregular data into R.
`----

This commit is revertable, and reverting this commit also solves the
described issue for me.

Cheers,
Andreas

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [babel] [BUG] table with header line as parameter
  2012-05-11 14:57 ` Andreas Leha
@ 2012-05-11 20:51   ` Eric Schulte
  0 siblings, 0 replies; 3+ messages in thread
From: Eric Schulte @ 2012-05-11 20:51 UTC (permalink / raw)
  To: Andreas Leha; +Cc: emacs-orgmode

Andreas Leha <andreas.leha@med.uni-goettingen.de> writes:

> Andreas Leha <andreas.leha@med.uni-goettingen.de> writes:
>
>> Hi all,
>>
>> I do not know what could be the cause of this, but I can't have a
>> table with header as argument to a source block any more:
>>
>> #+name: table_w_header
>> | one | two |
>> |-----+-----|
>> | 1   | 3   |
>>
>> #+begin_src R :var tbl=table_w_header
>>
>> #+end_src
>>
>> If I do C-c C-v v on the above source block, I get this backtrace:
>>
>> ,----
>> | Debugger entered--Lisp error: (wrong-type-argument sequencep hline)
>> |   length(hline)
>> |   mapcar(length (("one" "two") hline (1 3)))
>> |   org-babel-R-assign-elisp(tbl (("one" "two") hline (1 3)) nil nil)
>> |   #[(pair) "\302\b@\bA\303\304 \"A\305\232\303\306 \"A\305\232$\207"
>> | [pair params org-babel-R-assign-elisp assoc :colnames "yes"
>> | :rownames] 7]((tbl ("one" "two") hline (1 3)))
>> |   mapcar(#[(pair) "\302\b@\bA\303\304 \"A\305\232\303\306
>> | \"A\305\232$\207" [pair params org-babel-R-assign-elisp assoc
>> | :colnames "yes" :rownames] 7] ((tbl ("one" "two") hline (1 3))))
>> |   org-babel-variable-assignments:R(((:cache . "no") (:colname-names
>> | (tbl "one" "two")) (:comments . "") (:exports . "code") (:hlines
>> | . "no") (:noweb . "no") (:padline . "") (:padnewline . "yes")
>> | (:result-params "replace") (:result-type . value) (:results
>> | . "replace") (:rowname-names) (:session . "none") (:shebang . "")
>> | (:tangle . "no") (:var tbl (1 3))))
>> |   org-babel-expand-body:R("\n" ((:cache . "no") (:colname-names (tbl
>> | "one" "two")) (:comments . "") (:exports . "code") (:hlines . "no")
>> | (:noweb . "no") (:padline . "") (:padnewline . "yes")
>> | (:result-params "replace") (:result-type . value) (:results
>> | . "replace") (:rowname-names) (:session . "none") (:shebang . "")
>> | (:tangle . "no") (:var tbl (1 3))))
>> |   org-babel-expand-src-block()
>> |   call-interactively(org-babel-expand-src-block nil nil)
>> `----
>>
>> I tested on with emacs -Q on
>> 1. GNU Emacs 24.1.50.1 (x86_64-pc-linux-gnu, GTK+ Version 3.4.1) of 2012-05-02 on zelenka, modified by Debian
>> 2. GNU Emacs 23.4.1 (x86_64-pc-linux-gnu, GTK+ Version 2.24.10) of 2012-04-07 on trouble, modified by Debian
>>
>> My org-version: Org-mode version 7.8.09 with last commit
>> ,----
>> | commit 56520a6936351d6fcc149652e1ff30f50c43eb4e
>> | Merge: 505cc7a 0ebeb77
>> | Author: Nicolas Goaziou <n.goaziou@gmail.com>
>> | Date:   Wed May 9 23:34:05 2012 +0200
>> | 
>> |     Merge branch 'maint'
>> `----
>> So, the fix discussed in
>> [[https://lists.gnu.org/archive/html/emacs-orgmode/2011-02/msg01444.html]]
>> should be there.
>>
>>
>> Not sure if this could be connected:  I updated my emacs-snapshot (the
>> 24 version) from [[http://emacs.naquadah.org]] today.
>>
>>
>> Any help is highly appreciated,
>> Andreas
>
>
> Hi all,
>
> sorry for answering to my own message.
>
> I thought, that I did not pull between working state and broken state, but
> apparently I did.
>
> So, in the end, =git bisect= now told me the responsible commit:
> ,----
> | fb7ebd2dae66a7b42aecff695fe40461a33a76ed is the first bad commit
> | commit fb7ebd2dae66a7b42aecff695fe40461a33a76ed
> | Author: Eric Schulte <eric.schulte@gmx.com>
> | Date:   Wed Apr 25 15:34:15 2012 -0400
> | 
> |     clean up the code implementing reads of irregular data into R
> |     
> |     * lisp/ob-R.el (org-babel-R-assign-elisp): Clean up the code
> |       implementing reads of irregular data into R.
> `----
>
> This commit is revertable, and reverting this commit also solves the
> described issue for me.
>
> Cheers,
> Andreas
>
>

Hi Andreas,

Thanks for reporting and for bisecting (which made fixing much easier).
I've just pushed up a fix.

Please let me know if problems persist.

Best,

-- 
Eric Schulte
http://cs.unm.edu/~eschulte/

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-05-11 22:51 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-05-09 22:14 [babel] [BUG] table with header line as parameter Andreas Leha
2012-05-11 14:57 ` Andreas Leha
2012-05-11 20:51   ` Eric Schulte

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).