Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

new graphics array bug for legend #28359

Closed
kcrisman opened this issue Aug 15, 2019 · 3 comments
Closed

new graphics array bug for legend #28359

kcrisman opened this issue Aug 15, 2019 · 3 comments

Comments

@kcrisman
Copy link
Member

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!

CC: @egourgoulhon

Component: graphics

Issue created by migration from https://trac.sagemath.org/ticket/28359

@kcrisman kcrisman added this to the sage-8.9 milestone Aug 15, 2019
@egourgoulhon
Copy link
Member

comment:1

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.

@kcrisman
Copy link
Member Author

comment:2

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!

@egourgoulhon
Copy link
Member

comment:4

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants