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

Not all unicode works in plot (e.g. for axis labels) #21282

Closed
kcrisman opened this issue Aug 18, 2016 · 8 comments
Closed

Not all unicode works in plot (e.g. for axis labels) #21282

kcrisman opened this issue Aug 18, 2016 · 8 comments

Comments

@kcrisman
Copy link
Member

This answer to an ask.sagemath question point out that

plot(sin, 0, pi, axes_labels=('Ось абсцисс $x$', 'Ось ординат $y$'))

doesn't work. Apparently before #21008 was resolved it was worse. Maybe we can fix this?

CC: @jm58660 @fchapoton @paulmasson @slel

Component: graphics

Keywords: unicode, plot

Reviewer: Paul Masson

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

@kcrisman kcrisman added this to the sage-7.4 milestone Aug 18, 2016
@fchapoton
Copy link
Contributor

comment:1

This is purely a matplotlib problem with the default font. There is nothing to be fixed here. As explained in the doc of axes_labels in sage/plot/graphics.py, unicode characters are already accepted and transmitted correctly to matplotlib.

@kcrisman
Copy link
Member Author

comment:2

So ... but at the same time we have a confused user. And

Note that this does not guarantee that matplotlib will handle the strings properly, although it should.

Well, it didn't, apparently, in this case. So is this an upstream problem to report there? I think it would be reasonable to put this as an example that doesn't work in our doc, even, if upstream won't/can't fix it.

@slel
Copy link
Member

slel commented Jun 28, 2018

Changed keywords from none to unicode, plot

@slel
Copy link
Member

slel commented Jun 28, 2018

comment:4

Fixed by #21008?

@mkoeppe

This comment has been minimized.

@mkoeppe
Copy link
Member

mkoeppe commented Aug 17, 2020

comment:5

Adjusted example in the ticket description to py3 syntax

@mkoeppe
Copy link
Member

mkoeppe commented Aug 17, 2020

comment:6

Outdated, works for me

@mkoeppe mkoeppe removed this from the sage-7.4 milestone Aug 17, 2020
@paulmasson
Copy link
Mannequin

paulmasson mannequin commented Aug 17, 2020

Reviewer: Paul Masson

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

4 participants