#28359 closed defect (fixed)

new graphics array bug for legend

Reported by: kcrisman Owned by:
Priority: minor Milestone: sage-duplicate/invalid/wontfix
Component: graphics Keywords:
Cc: egourgoulhon Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

Thank you to everyone for #27865!

However, I think there are unintended consequences when it comes to certain types of overlapping options. Can we set the options individually?

P1 = plot(prime_pi,3,30,color='black',legend_label='1')
#P1 += plot(Li,3,30, color='green',legend_label='Li1')
P2 = plot(prime_pi,40,70,color='blue',legend_label='2')
#P2 += plot(Li,40,70, color='red',legend_label='Li2')
graphics_array([P1,P2])

This doubles the labels in the second graphic. If you uncomment the other lines more things happen, though it might all be just one bug. Trying graphics_array([P1,P2,P2]) only doubles the last one.

I believe this is new since the new graphics array stuff, though I can't be sure. Thanks for looking into it!

Change History (5)

comment:1 Changed 11 months ago by egourgoulhon

Are you using a version of Sage recent enough to include #27865 (i.e. >= 8.9.beta0)? I've tried with Sage 8.9.beta6 and everything is fine: the legend labels are not doubled. Uncommenting the other lines leads to a correct result as well.

comment:2 follow-up: Changed 11 months ago by kcrisman

  • Milestone changed from sage-8.9 to sage-duplicate/invalid/wontfix
  • Status changed from new to needs_review

Oops! I have a local install that is updated, but I found this using the Sage cell server, which is of course still not-necessarily-updated - but which I was using to test some auto-generated images before actually generating them. Sorry!

comment:3 Changed 11 months ago by kcrisman

  • Status changed from needs_review to positive_review

comment:4 in reply to: ↑ 2 Changed 11 months ago by egourgoulhon

Replying to kcrisman:

Oops! I have a local install that is updated, but I found this using the Sage cell server, which is of course still not-necessarily-updated - but which I was using to test some auto-generated images before actually generating them. Sorry!

No problem (it is always nice to have extra checks). Indeed, the Sage Cell server is running Sage 8.8, while #27865 was included only in 8.9.beta0.

comment:5 Changed 11 months ago by chapoton

  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.