bugGNU Octave - Bugs: bug #49341, demo legend 14 produce error with...

 
 

bug #49341: demo legend 14 produce error with gnuplot

Submitted by:  Avinoam Kalma <avinoam>
Submitted on:  Fri 14 Oct 2016 09:12:39 AM UTC  
 
Category: Plotting with gnuplotSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Regression
Status: FixedAssigned to: None
Originator Name: AvinoamOpen/Closed: Closed
Release: devOperating System: Any

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

(Jump to the original submission Jump to the original submission)

Sun 16 Oct 2016 07:26:54 PM UTC, comment #9:

I tweaked the hggroup patch slightly

to follow some Octave coding guidelines.

It was applied here http://hg.savannah.gnu.org/hgweb/octave/rev/5ab3c91fc4bb.

Fixed, closing report.

Rik <rik5>
Project Administrator
Sun 16 Oct 2016 10:38:29 AM UTC, comment #8:

Applying this patch fixed the error in legend 14, as well in other demos

Avinoam Kalma <avinoam>
Project Member
Sun 16 Oct 2016 07:33:16 AM UTC, comment #7:

Attached is a fix for the legend entry of some more hggroup objects. I've done what I can as far as almost-correct legend symbols. One of gnuplot's limitations is in the area of keys and key entries. I wish it was done differently however many years ago now. In order to construct a key with both lines and points, someone devised a new style 'linespoints': sounds good, but it is so restrictive once other styles are introduced, e.g., error bars and boxes, etc. Then try remember the names. It would have been much nicer if one could specify "plot <mydata> with lines ... and points ..." such that all of those items appear in the key symbol.

Anyway, try the following:

demo legend 17
demo legend 18
demo legend
demo errorbar

The stem series of legend 17 demo illustrates how a circle and line can be in the key IF the two are the same color. But linespoints can't have different colors for lines and for points, so those two have to be done separately--hence cannont be in the same key symbol.

(file #38738)

Dan Sebald <sebald>
Sat 15 Oct 2016 04:34:39 PM UTC, comment #6:

Forgot attachment...

(file #38736)

Dan Sebald <sebald>
Sat 15 Oct 2016 04:33:06 PM UTC, comment #5:

OK, that's me.

Attached is a change to demo 17 which adds a second subplot with another hggroup, stems. (The error bars hggroup is covered well by the errorbar.m demos.) I think those are all the hggroup object collections.

The patch doesn't fix anything as far as bugs, but it brings out a couple issues in both OpenGL and gnuplot toolkits. For OpenGL, the legend placement doesn't seem quite right in subplots and there is an axes border for only one of the subplots. For gnuplot, the legends are wrong.

By the way, for stems the proper legend graphic is simply the stem marker, is that correct? There is no other line in the legend like might be done for error bars?

Dan Sebald <sebald>
Sat 15 Oct 2016 09:51:52 AM UTC, comment #4:

@Dan: I pushed your patch for legend demo 23 here (http://hg.savannah.gnu.org/hgweb/octave/rev/37b7b86f62f2).

The name change to autopos_tag took place on the development branch. I have no problems running legend demos 26, 27, 28. You might need to clean out cruft with 'make distclean' and then re-run configure and make.

Rik <rik5>
Project Administrator
Sat 15 Oct 2016 07:58:00 AM UTC, comment #3:

OK, I see this. There are multiple hggroup regression problems here. I will find a fix for that on the development branch.

On the stable branch there are a couple of general problems with legend demos that should be fixed beyond the above. Attached is a fix for a typo in demo 23.

Also, Rik could you have a look at demos 26, 27 and 28:

I see there is extensive use of property 'autopos_tag' in the C++ file "libinterp/corefcn/graphics.cc. Is this 'autopos_tag' one of the hidden variables that was changes to _autopos_tag_?

(file #38734)

Dan Sebald <sebald>
Fri 14 Oct 2016 02:08:37 PM UTC, comment #2:

Confirmed. I added a dependency on bug #49260 which is what the original cset was trying to fix.

Rik <rik5>
Project Administrator
Fri 14 Oct 2016 11:20:47 AM UTC, comment #1:

The problem is in line 711 of _gnuplot_draw_axis_, and it is only in the dev branch, because it is caused by
http://hg.savannah.gnu.org/hgweb/octave/rev/54332b6ddeb6 which wasn't applied to the stable branch.

Avinoam Kalma <avinoam>
Project Member
Fri 14 Oct 2016 09:12:39 AM UTC, original submission:

It is marked as regression because in in 4.2-rc2 there is n such problem

Avinoam Kalma <avinoam>
Project Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

 

Digest:
   bug dependencies.

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by sebald (Updated the item)
  • -unavailable- added by rik5 (Posted a comment)
  • -unavailable- added by avinoam (Submitted the item)
  • -unavailable- added by avinoam
  • -unavailable- added by avinoam
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 10 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sun 16 Oct 2016 07:26:54 PM UTCrik5StatusConfirmed=>Fixed
      Open/ClosedOpen=>Closed
    Sun 16 Oct 2016 07:33:16 AM UTCsebaldAttached File-=>Added octave-gnuplot_hggroup_legend_bug49341-djs2016oct16.patch, #38738
    Sat 15 Oct 2016 04:34:39 PM UTCsebaldAttached File-=>Added octave-legend_demo_17_bug49341-djs2016oct15.patch, #38736
    Sat 15 Oct 2016 07:58:00 AM UTCsebaldAttached File-=>Added octave-legend_demo_23_bug49341-djs2016oct15.patch, #38734
    Fri 14 Oct 2016 02:08:37 PM UTCrik5StatusNone=>Confirmed
    Fri 14 Oct 2016 02:07:02 PM UTCrik5Dependencies-=>Depends on bugs #49260
    Fri 14 Oct 2016 11:20:47 AM UTCavinoamOperating SystemMicrosoft Windows=>Any
    Fri 14 Oct 2016 09:12:39 AM UTCavinoamCarbon-Copy-=>Added -unavailable-
      Carbon-Copy-=>Added -unavailable-

    Back to the top


    Powered by Savane 3.1-cleanup1