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

Submitted by:  None
Submitted on:  Mon 24 Nov 2014 08:23:57 PM UTC  
 
Category: PlottingSeverity: 2 - Minor
Priority: 5 - NormalItem Group: Matlab Compatibility
Status: FixedAssigned to: Pantxo Diribarne <pantxo>
Originator Name: Originator Email: -unavailable-
Open/Closed: ClosedRelease: 3.8.1
Operating System: GNU/Linux

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

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>
Project MemberIn charge of this item.
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>
Project Administrator
Thu 27 Nov 2014 08:54:44 PM UTC, comment #3:

Thanks Michael, I attached a simple patch.

(file #32562)

Pantxo Diribarne <pantxo>
Project MemberIn charge of this item.
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>
Project 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?

Pantxo

Pantxo Diribarne <pantxo>
Project MemberIn charge of this item.
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 File(s):
   
   
Comment:
   

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by rik5 (Posted a comment)
  • -unavailable- added by godfrey (Posted a comment)
  • -unavailable- added by pantxo (Posted a comment)
  •  

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

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 01 Dec 2014 08:56:51 PM UTCpantxoStatusPatch Submitted=>Fixed
      Open/ClosedOpen=>Closed
    Mon 01 Dec 2014 07:10:36 PM UTCrik5Summaryquiver 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
    Mon 01 Dec 2014 05:00:57 PM UTCrik5Severity3 - Normal=>2 - Minor
      StatusNone=>Patch Submitted
      Assigned toNone=>pantxo
    Thu 27 Nov 2014 08:54:44 PM UTCpantxoAttached File-=>Added quiver_initialize_uu.patch, #32562

    Back to the top


    Powered by Savane 3.1-cleanup1