[Octave-bug-tracker] [bug #43282] annotation function missing

[Pantxo Diribarne]

Follow-up Comment #15, bug #43282 (project octave):

@John: I am ok with pushing the patch as is, can't do it my self so go ahead.
I'll be following the bug tracker during the integration process anyway.

FTR, here is (I hope) the missing part:


Without patch
1 axes:         0.01 s
2 axes:         0.02 s
4 axes:         0.03 s
8 axes:         0.07 s
16 axes:        0.14 s
32 axes:        0.27 s   

With patch
1 axes:         0.01 s
2 axes:         0.02 s
4 axes:         0.04 s
8 axes:         0.09 s
16 axes:        0.18 s
32 axes:        0.40 s


There is indeed a performance hit for a large number of axes in a figure. In
the mean time 30 axes in a figure is quite large. 

I also noticed that if annotation function is called first in a figure, it
becomes the currentaxes which should not happen. I updated annotation.m to
take this into account. 
 
@Michael: depends on when feature freeze is declared. Is there a proposed
date? I have no time now to cleanup the new patch but I may be able to post
it
in the beginning of next week. Nevertheless if John wants to work on
integration, it will probably look like (or be) the attached
add_annotation2.patch.  

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?43282>

_______________________________________________
  Message posté via/par Savannah
  http://savannah.gnu.org/