bugGNU Octave - Bugs: bug #43686, quiver doesn't handle case where 2...

 
 

bug #43686: quiver doesn't handle case where 2 zero matrices are passed in as the vector fields

Submitter:  None
Submitted:  Mon 24 Nov 2014 08:23:57 PM UTC
   
 
Category:  Plotting Severity:  2 - Minor
Priority:  5 - Normal Item Group:  Matlab Compatibility
Status:  Fixed Assigned to:  pantxo
Originator Name:  Originator Email:  -email is unavailable-
Open/Closed:  * Closed Release:  * 3.8.1
Operating System:  * GNU/Linux Fixed Release:  None
Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 01 Dec 2014 08:56:51 PM UTC, comment #5: 

Hi,

I pushed the patch at http://hg.savannah.gnu.org/hgweb/octave/rev/009191dbc76d

Closing bug report.

Pantxo Diribarne <pantxo>
Group Member
Mon 01 Dec 2014 05:00:57 PM UTC, comment #4: 

@Pantxo: The patch seems fine.  Why don't you apply it to the development branch of Octave and close the report.

Rik <rik5>
Group administrator
Thu 27 Nov 2014 08:54:44 PM UTC, comment #3: 

Thanks Michael, I attached a simple patch.

(file #32562)

Pantxo Diribarne <pantxo>
Group Member
Wed 26 Nov 2014 10:44:44 AM UTC, comment #2: 

In my pretty old Matlab:

>> version


ans =

7.9.0.529 (R2009b)

>> h = quiver (zeros(2), zeros(2));
>> nkids = numel (get (h, 'children'))


nkids =

     3

And it draws a blank figure wirh scale: 1-2
for x and y.

Michael Godfrey <godfrey>
Group Member
Tue 25 Nov 2014 11:29:14 PM UTC, comment #1: 

Hi,

Thanks for your report. If you have access to Matlab can you tell us how it handles this particular situation? Does it raise a warning? Are the line objects even created in the example below? 


h = quiver (zeros(2), zeros(2));
nkids = numel (get (h, 'children'))


Pantxo

Pantxo Diribarne <pantxo>
Group Member
Mon 24 Nov 2014 08:23:57 PM UTC, original submission:  

if you try plotting say quiver(zeros(5),zeros(5)) it gives the error

error: 'uu' undefined near line 196 column 16
error: called from:
error:   /usr/share/octave/3.8.1/m/plot/draw/private/__quiver__.m at line 196, column 10
error:   /usr/share/octave/3.8.1/m/plot/draw/quiver.m at line 80, column 10

which is uninformative and makes it difficult when swapping pde solvers that plot a vector field from matlab to octave, this led to a few hours of annoyance as I had to figure out what uu was and why it was undefined. Please fix small error but common when solving pde's or ode's.

Anonymous

 

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

Attach Files:
   
   
Comment:
   

Attached Files

 

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 godfrey (Posted a comment)
  • -email is unavailable- added by pantxo (Posted a comment)
  •  

    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 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2014-12-01 pantxo StatusPatch Submitted Fixed
        Open/ClosedOpen Closed
    2014-12-01 rik5 Summaryquiver doesn\'t handle case where 2 zero matricies are passed in as the vector fields quiver doesn't handle case where 2 zero matrices are passed in as the vector fields
    2014-12-01 rik5 Severity3 - Normal 2 - Minor
        StatusNone Patch Submitted
        Assigned toNone pantxo
    2014-11-27 pantxo Attached File- Added quiver_initialize_uu.patch, #32562

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code