[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [BUG] org-plot: Unable to use text xtics with type:2d (+ more) [9.7-
From: |
Visuwesh |
Subject: |
Re: [BUG] org-plot: Unable to use text xtics with type:2d (+ more) [9.7-pre (N/A @ /home/viz/lib/emacs/straight/build/org/)] |
Date: |
Mon, 17 Jun 2024 10:36:17 +0530 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
[சனி ஜூன் 15, 2024] Ihor Radchenko wrote:
> Visuwesh <visuweshm@gmail.com> writes:
>
>> Sorry for the noise, I copied the wrong link in the commit message.
>> Please see attached instead.
>
> Thanks!
> I have some comments.
>
>> - (type (assoc (plist-get params :plot-type)
>> - org-plot/preset-plot-types))
>> + (type (cdr (assoc (plist-get params :plot-type)
>> + org-plot/preset-plot-types)))
>> gnuplot-script)
>
> This may break the existing customization.
> Later in the function, TYPE is used as an argument for
> `org-plot/gnuplot-term-extra' and `org-plot/gnuplot-script-preamble'.
> Some users may have these two custom options adjusted to the older
> calling convention.
>
> To not break things, we should pass the full `assoc' to these functions.
If you meant org-plot/gnuplot-script eventually calling these functions,
then I don't see how the above change will break things. Even in
org-plot/gnuplot-script, TYPE passed to both these user options are
(let* ((type-name (plist-get params :plot-type))
(type (cdr (assoc type-name org-plot/preset-plot-types))))
so there should be no harm done by the above change since TYPE is not an
argument taken by org-plot/gnuplot-script.
> Also, while you are at it, may your please clarify what TYPE means in
> the docstrings of `org-plot/gnuplot-term-extra' and
> `org-plot/gnuplot-script-preamble'?
OK.
>> (setf params (org-plot/collect-options params))))
>> + (setq params (org-combine-plists type params))
>
> May you also drop a short comment in the code that explains what this
> line does?
I will send an updated patch once my confusion is cleared.
- Re: [BUG] org-plot: Unable to use text xtics with type:2d (+ more) [9.7-pre (N/A @ /home/viz/lib/emacs/straight/build/org/)], Visuwesh, 2024/06/11
- Re: [BUG] org-plot: Unable to use text xtics with type:2d (+ more) [9.7-pre (N/A @ /home/viz/lib/emacs/straight/build/org/)], Ihor Radchenko, 2024/06/12
- Re: [BUG] org-plot: Unable to use text xtics with type:2d (+ more) [9.7-pre (N/A @ /home/viz/lib/emacs/straight/build/org/)], Visuwesh, 2024/06/13
- Re: [BUG] org-plot: Unable to use text xtics with type:2d (+ more) [9.7-pre (N/A @ /home/viz/lib/emacs/straight/build/org/)], Visuwesh, 2024/06/13
- Re: [BUG] org-plot: Unable to use text xtics with type:2d (+ more) [9.7-pre (N/A @ /home/viz/lib/emacs/straight/build/org/)], Ihor Radchenko, 2024/06/15
- Re: [BUG] org-plot: Unable to use text xtics with type:2d (+ more) [9.7-pre (N/A @ /home/viz/lib/emacs/straight/build/org/)],
Visuwesh <=
- Re: [BUG] org-plot: Unable to use text xtics with type:2d (+ more) [9.7-pre (N/A @ /home/viz/lib/emacs/straight/build/org/)], Ihor Radchenko, 2024/06/17
- Re: [BUG] org-plot: Unable to use text xtics with type:2d (+ more) [9.7-pre (N/A @ /home/viz/lib/emacs/straight/build/org/)], Visuwesh, 2024/06/18
- Re: [BUG] org-plot: Unable to use text xtics with type:2d (+ more) [9.7-pre (N/A @ /home/viz/lib/emacs/straight/build/org/)], Ihor Radchenko, 2024/06/18