bugGNU Octave - Bugs: bug #59388, legend with error boxes shows...

 
 

bug #59388: legend with error boxes shows error bars

Submitter:  Markus Mützel <mmuetzel>
Submitted:  Sun 01 Nov 2020 11:00:33 AM UTC
   
 
Category:  Plotting Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Incorrect Result
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * 6.0.92 Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Mon 02 Nov 2020 09:53:28 PM UTC, comment #7: 

I pushed the patch.  Marking as fixed and closing report.

Rik <rik5>
Group administrator
Mon 02 Nov 2020 08:40:30 PM UTC, comment #6: 

I attached an updated patch which now falls-back to drawing a line.

(file #50195)

Pantxo Diribarne <pantxo>
Group Member
Mon 02 Nov 2020 08:16:12 PM UTC, comment #5: 

I see now that the hggroup is using labels "boxx", "boxy", "boxxy" so the single decode for "box" will work.

Rik <rik5>
Group administrator
Mon 02 Nov 2020 08:12:46 PM UTC, comment #4: 

I like the idea of defensively programming and having a default case.  Also, the if/elseif tree covers 4 cases: "xerr", "yerr", "xyerr", and "box".  Does it neet to cover "xbox", "ybox", and "xybox"?

I changed the release to 6.0.92.  This is just a bug and should be fixed at any time, including before the release.

Rik <rik5>
Group administrator
Mon 02 Nov 2020 07:46:50 AM UTC, comment #3: 

Thanks for the quick patch.
Haven't tested yet. But should we maintain an fall-back for the case where none of the if or elseif branches match? (E.g. let's assume that at some point in the future we'll add "error ellipses".)
Should the last "elseif" be just an "else"?

Markus Mützel <mmuetzel>
Group administrator
Sun 01 Nov 2020 10:57:11 PM UTC, comment #2: 

The patch:

(file #50186)

Pantxo Diribarne <pantxo>
Group Member
Sun 01 Nov 2020 10:56:36 PM UTC, comment #1: 

The attached patch changes that icon to be a box for all box formats.

Pantxo Diribarne <pantxo>
Group Member
Sun 01 Nov 2020 11:00:33 AM UTC, original submission:  

Running `demo errorbar 5`, the legend for both lines shows error bars.
The legend for the second line should probably show error boxes (like the corresponding line).

The commands for `demo errorbar 5`:

 clf;
 x = 0:0.5:2*pi;
 err = x/30;
 y1 = sin (x);
 y2 = cos (x);
 errorbar (x, y1, err, err, err, err, "~>", ...
           x, y2, err, err, err, err, "#~>-*");
 legend ("X-Y errbars", "X-Y errboxes");
 title ("errorbar() with X-Y errorbars and error boxes");


Markus Mützel <mmuetzel>
Group administrator

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #50195:  bug59388_2.patch added by pantxo (1KiB - text/x-patch)
file #50186:  bug59388.patch added by pantxo (1KiB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by pantxo (Posted a comment)
  • -email is unavailable- added by mmuetzel (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2020-11-02 rik5 StatusPatch Submitted Fixed
        Open/ClosedOpen Closed
    2020-11-02 pantxo Attached File- Added bug59388_2.patch, #50195
    2020-11-02 rik5 Releasedev 6.0.92
    2020-11-01 pantxo Attached File- Added bug59388.patch, #50186
    2020-11-01 pantxo StatusNone Patch Submitted

    Back to the top

    Powered by Savane 3.13-4b48.
    Corresponding source code