[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] ob-calc: Format vector and matrix results as Org table
From: |
Ihor Radchenko |
Subject: |
Re: [PATCH] ob-calc: Format vector and matrix results as Org table |
Date: |
Tue, 24 Dec 2024 10:58:12 +0000 |
Visuwesh <visuweshm@gmail.com> writes:
> Attached patch formats vector and matrix results from a Calc source
> block as Org table. To test, try
> ...
> now. The patch passes the old tests, and I added three new tests to
> check the correctness of this conversion.
Thanks!
etc/ORG-NEWS entry seems to be displaced. It should be under Org 9.8
heading.
Also, the patch does not apply onto the latest main.
> -<point>#+BEGIN_SRC calc :results silent :var a=ob-calc-table-1
> +<point>#+BEGIN_SRC calc :results silent verbatim :var a=ob-calc-table-1
You had to change the tests, so we have a breaking change.
The change can be viewed as a bug fix since ob-calc previously did not
conform to babel conventions, so I do not object changing the defaults.
However, we need to inform the affected users and put the announcement
into "breaking changes" section. We should also recommend changing
org-babel-default-header-args:calc to restore the old behavior.
--
Ihor Radchenko // yantar92,
Org mode maintainer,
Learn more about Org mode at <https://orgmode.org/>.
Support Org development at <https://liberapay.com/org-mode>,
or support my work at <https://liberapay.com/yantar92>