Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

fix for #3669 Font issue without PyCXX#3671

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
mdboom merged 1 commit intomatplotlib:masterfromjenshnielsen:fix_3669
Oct 20, 2014

Conversation

@jenshnielsen
Copy link
Member

Thisfixes#3669 for me. It looks like CharacterComplement doesn't return a valid unicode string so make it a bytestring in python3

This happens to me for the standard matplotlib font matplotlib/mpl-data/fonts/ttf/Vera.ttf
not sure why it doesn't happen on travis.

In any case 'characterComplement' is '\xff\xff\xff\xff6\xff\xff\xfe628R00' for this font on both python2 and python3 which looks rather strange to me. I expected this to be a string of all chars that this font contains so perhaps there is a deeper issue?

Looks like CharacterComplement doesn't return a unicode string so make it a bytestring in python3
@tacaswelltacaswell added this to thev1.5.x milestoneOct 18, 2014
@mdboom
Copy link
Member

Looks good. Thanks!

mdboom added a commit that referenced this pull requestOct 20, 2014
@mdboommdboom merged commit4aa6b7e intomatplotlib:masterOct 20, 2014
@mdboom
Copy link
Member

characterComplement is essentially a bitfield. See here:

http://www.microsoft.com/typography/otspec/pclt.htm

If we wanted to provide a really Pythonic API to it, it would probably be a list of bools or something. But matplotlib itself doesn't even use this, and it's part of matplotlib's public API so I didn't want to change it (well, not intentionally).

@jenshnielsen
Copy link
MemberAuthor

Thanks for the explanation. I didn't manage to google a good explanation of CharacterComplement

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v2.1

Development

Successfully merging this pull request may close these issues.

Test faliures after merging the decxx branch (#3646)

4 participants

@jenshnielsen@mdboom@tacaswell@QuLogic

[8]ページ先頭

©2009-2025 Movatter.jp