Re: Possible Calc support for Org-Babel?

["Eric Schulte" <schulte.eric@gmail.com>]

Eric S Fraga <ucecesf@ucl.ac.uk> writes:

> "Eric Schulte" <schulte.eric@gmail.com> writes:
>
>> Eric S Fraga <ucecesf@ucl.ac.uk> writes:
>>
>>> Eric,
>>>
>>> This is really nice!
>>>
>>> I had a problem initially in that calc-push-list was undefined.  I had
>>> to initiate calc first so maybe a
>>>
>>> : (require 'calc)
>>>
>>
>> Ah yes, that is in the ob-calc.el file, but not in the code snippet I
>> shared.  I'll commit this entire file to the repository.
>
> Ah, so should I use the code snippet you sent or should I be requiring
> 'ob-calc?  In either case, I still have problems (see below).
>

Require ob-calc, please don't use the snippet below.

>
>>> is required to ensure the functions you use are available?
>>>
>>> Then, out of the three examples you give, only one (3^3) worked.  The
>>> others give me:
>>>
>>
>> I just made a change which should fix this issue.
>
> Very strange.  I'm still seeing the same problems, both with
> calc-push-list not known (having to manually require 'calc) and then
> errors evaluating the 1+2 block:
>

judging from the error below, it looks as though you are still using the
old code (or the snippet below).  Please ensure you're on the latest and
give it another go.

>
> Debugger entered--Lisp error: (wrong-type-argument stringp (2
>"Expected a number"))
>   string-match("\\` *\\([0-9]+\\) *\\'" (2 "Expected a number"))
>   math-read-number((2 "Expected a number"))
>   (list (math-read-number (calc-eval line)))

in the latest code the line

  (list (math-read-number (calc-eval line)))

has been replaced by

  (list ((lambda (res)
    (if (numberp res) res (math-read-number res)))
   (calc-eval line)))

Best -- Eric