|
From: Philipp K. J. <ja...@ie...> - 2015-01-08 00:09:02
|
I seem to have observed three minor bugs in gp5:
1) Filledcurve above/below
My understanding is that
plot sin(x) w filledc above y1=0
should fill only the area between y=0 and
the sine curve. What I observe is that it
does not fill at all. With
plot sin(x) w filledc below y1=0
gnuplot fills both above and below.
The plot is sensitive to the value supplied,
but above/below do not seem to be interpreted
properly.
2) Enhanced text, key, and cairo
set key title "{/:Bold Hello}"
plot sin(x) t "World"
I'd expect the title of the key to be bold,
but the title entry for the curve to be
normal weight. That's also what it does in
wxt, but for any of the cairo terminals, all
key entries are bold.
3) set xmtics, save, load
(I agree, a truly dusty corner!)
set xmtics
plot "data" u 1:2 w l
save "foobar.gp"
load "foobar.gp"
This produces an error:
set xmtics norangelimit
unexpected or unrecognized token
No biggy all three, but I'd thought I'd point them
out. Maybe worth a remark in the release notes.
|
|
From: Petr M. <mi...@ph...> - 2015-01-08 12:57:22
|
> I seem to have observed three minor bugs in gp5: > > 1) Filledcurve above/below > > My understanding is that > plot sin(x) w filledc above y1=0 > should fill only the area between y=0 and > the sine curve. What I observe is that it > does not fill at all. With > plot sin(x) w filledc below y1=0 > gnuplot fills both above and below. > > The plot is sensitive to the value supplied, > but above/below do not seem to be interpreted > properly. I see, the above/below keywords work correctly in 4.6 but not in 5.0. I think it was not spotted because fillcrvs.dem does not test them. --- PM |
|
From: Philipp K. J. <ja...@ie...> - 2015-01-08 22:03:31
|
On Thu, 8 Jan 2015 13:56:29 +0100 (CET) Petr Mikulik <mi...@ph...> wrote: > > I seem to have observed three minor bugs in gp5: > > > > 1) Filledcurve above/below > > > > My understanding is that > > plot sin(x) w filledc above y1=0 > > should fill only the area between y=0 and > > the sine curve. What I observe is that it > > does not fill at all. With > > plot sin(x) w filledc below y1=0 > > gnuplot fills both above and below. > > > > The plot is sensitive to the value supplied, > > but above/below do not seem to be interpreted > > properly. > > I see, the above/below keywords work correctly in 4.6 but not in 5.0. > I think it was not spotted because fillcrvs.dem does not test them. Interestingly, it works if the limiting line is provided as "data". This command works as advertised: plot "+" u 1:(sin($1)):(0) w filledc above > > --- > PM |
|
From: sfeam <sf...@us...> - 2015-01-09 01:40:12
|
On Wednesday, 07 January 2015 04:08:50 PM Philipp K. Janert wrote:
>
> I seem to have observed three minor bugs in gp5:
>
> 2) Enhanced text, key, and cairo
>
> set key title "{/:Bold Hello}"
> plot sin(x) t "World"
>
> I'd expect the title of the key to be bold,
> but the title entry for the curve to be
> normal weight. That's also what it does in
> wxt, but for any of the cairo terminals, all
> key entries are bold.
That odd, because wxt _is_ one of the cairo terminals.
It seems the specific bug is that the final curly bracket in
the key title doesn't take effect if it is the last character
in the string. The problem goes away if you add a blank,
or anything else, after the closing bracket.
That must be a big clue, but at the moment I don't
see where it points. Most of the code involved is
shared by all terminals, so why does the problem only
show up in a couple of them?
Ethan
|
|
From: Jun T. <tak...@kb...> - 2015-01-15 13:57:21
|
On 2015/01/09, at 10:36, sfeam <sf...@us...> wrote:
> On Wednesday, 07 January 2015 04:08:50 PM Philipp K. Janert wrote:
>>
>> I seem to have observed three minor bugs in gp5:
>>
>> 2) Enhanced text, key, and cairo
>
> That odd, because wxt _is_ one of the cairo terminals.
(I know a patch is already in CVS)
In cairotrm_put_text(), enhanced_recursion() is used only if the string
contains mark-up characters "{}^_@&~"; otherwise gp_cairo_draw_text() is used.
When drawing "{/:Bold Hello}",
enhanced_recursion() calls gp_cairo_enhanced_open(),
which calls gp_cairo_set_font(),
which sets plot->fontweight (gp_cairo.cpp:line 324):
plot->fontweight = PANGO_WEIGHT_BOLD;
But this fontweight is not reset to the original value after drawing
the string.
When drawing "World",
gp_cairo_draw_text() uses the plot->fontweight (gp_cairo.cpp:line 867),
which still has the value PANGO_WEIGHT_BOLD:
pango_font_description_set_weight (desc, plot->fontweight);
In wxt_put_text() (wxt_gui.cpp), on the other hand, enhanced_recursion()
is used always; thus wxt does not have this problem.
Jun
|
|
From: Ethan A M. <sf...@us...> - 2015-01-15 17:40:17
|
On Thursday, 15 January, 2015 22:13:14 Jun T. wrote:
>
> On 2015/01/09, at 10:36, sfeam <sf...@us...> wrote:
>
> > On Wednesday, 07 January 2015 04:08:50 PM Philipp K. Janert wrote:
> >>
> >> I seem to have observed three minor bugs in gp5:
> >>
> >> 2) Enhanced text, key, and cairo
> >
> > That odd, because wxt _is_ one of the cairo terminals.
>
>
> (I know a patch is already in CVS)
>
> In cairotrm_put_text(), enhanced_recursion() is used only if the string
> contains mark-up characters "{}^_@&~"; otherwise gp_cairo_draw_text() is used.
> [snip]
> In wxt_put_text() (wxt_gui.cpp), on the other hand, enhanced_recursion()
> is used always; thus wxt does not have this problem.
Thank you. Yes, that indeed explains the difference.
Ethan |
|
From: sfeam <sf...@us...> - 2015-01-17 06:04:10
|
On Wednesday, 07 January 2015 04:08:50 PM Philipp K. Janert wrote: > > I seem to have observed three minor bugs in gp5: > > > 1) Filledcurve above/below > [snip] > 2) Enhanced text, key, and cairo > [snip] > 3) set xmtics, save, load > [snip] These are now fixed. Ethan |
|
From: Philipp K. J. <ja...@ie...> - 2015-01-18 05:34:51
|
On Fri, 16 Jan 2015 22:00:27 -0800 sfeam <sf...@us...> wrote: > On Wednesday, 07 January 2015 04:08:50 PM Philipp K. Janert wrote: > > > > I seem to have observed three minor bugs in gp5: > > > > > > 1) Filledcurve above/below > > [snip] > > 2) Enhanced text, key, and cairo > > [snip] > > 3) set xmtics, save, load > > [snip] > > These are now fixed. Awesome! ;-) |