[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Octave-task-tracker] [task #14243] Rewrite legend.m

From: Pantxo Diribarne
Subject: [Octave-task-tracker] [task #14243] Rewrite legend.m
Date: Tue, 12 Nov 2019 06:56:57 -0500 (EST)
User-agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:69.0) Gecko/20100101 Firefox/69.0

Follow-up Comment #14, task #14243 (project octave):

@Rik: Here is an updated patch.
* Case #1: Now ok;
* Case #2: That is not what I observe when testing online: Matlab adds a
legend with as many item as plots in the axes. In ML, the legend then extends
below the figure limits while in Octave it is layout in two columns to fit in
the axes box.
* Case #3: I'd like to remove this possibility of specifying the location
through numeric values
* Case #3 bis: Now ok;
* Case #4, 5, 6, 7, 8, 9, 10, 11, 12: Now ok;
* Case #13, 14: Yes, I still have to find a reliable way to handle plotyy; The
new patch improves the situation.
* Case #15: I don't know how to handle outside legends in subplots: by default
subplots are supposed to be aligned, which is what the listeners in subplot.m
try to ensure. This is the reason why the narrowest axes (due to its long
legend) dictates the size of other axes. This looks consistent to me (and
straightforward) and I would like to avoid having to make the subplot.m code
"legend aware". Note that previously in Octave, outside legends would simplpy
break the subplot auto-alignment feature, which is not better...
* Case #16: I added back demo 36
* Case #17: That is not what I observe, the average after running demo 14 ten
times is roughly the same: New code 0.265s, Old code 0.280s.
* Case #18: I reduced the item size to 15pts
* Case #19: I agree the __appdata__ should be hidden, but I think this is
outside the scope of this task and I'd prefer to handle that in a separate bug
* Case #20: Now ok.

Finally the new patch uses an additional line object to center the markers.

(file #47841)

Additional Item Attachment:

File name: task14243_legend_3.patch       Size:156 KB


Reply to this item at:


  Message posté via Savannah

reply via email to

[Prev in Thread] Current Thread [Next in Thread]