bugGNU Octave - Bugs: bug #38927, fill(x, y, "EdgeColor",...

 
 

bug #38927: fill(x, y, "EdgeColor", "red")

Submitter:  Muhali <muhali>
Submitted:  Tue 07 May 2013 01:31:20 PM UTC
   
 
Category:  Plotting with OpenGL Severity:  4 - Important
Priority:  5 - Normal Item Group:  None
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * dev Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Tue 08 Oct 2013 10:53:52 PM UTC, comment #2: 

I applied your patch, with a few minor style tweaks, here http://hg.savannah.gnu.org/hgweb/octave/rev/ec31ca0a5812.

I followed it with a patch that changes the facecolor property from "interp" to "flat" based on the orientation of the color data vector since that is what Matlab does (http://hg.savannah.gnu.org/hgweb/octave/rev/3f8b3588a9f0).

Closing report.

Rik <rik5>
Group administrator
Thu 09 May 2013 02:26:12 PM UTC, comment #1: 

According to the documentation (also tested in Matlab) this example is not supposed to work. fill expects at least 3 arguments before the option/value pairs. The function should thus send an error.

Attached is a patch that fixes the input checking and avoids the endless loop.

(file #28054)

Pantxo Diribarne <pantxo>
Group Member
Tue 07 May 2013 01:31:20 PM UTC, original submission:  

When using fill with edgecolor, as in


fill([0 0 1 1], [0 1 1 0], "EdgeColor", "red") ;


octave stalls. Same with e.g. "EdgeColo" (no error message).

This is with OpenGL. With gnuplot I get

error: _go_draw_axes_: A(I): index out of bounds; value 1 out of bound 0

Muhali <muhali>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #28054:  fill_fixinputcheck.patch added by pantxo (3KiB - 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 (Updated the item)
  • -email is unavailable- added by pantxo (Updated the item)
  • -email is unavailable- added by muhali (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 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-10-08 rik5 StatusNone Fixed
        Open/ClosedOpen Closed
    2013-10-08 rik5 Severity3 - Normal 4 - Important
    2013-05-09 pantxo Attached File- Added fill_fixinputcheck.patch, #28054

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code