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

Try to install the Noto Sans CJK font#20851

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
anntzer merged 1 commit intomatplotlib:masterfromjkseppan:wqy-zenhei-install
Aug 30, 2021

Conversation

jkseppan
Copy link
Member

@jkseppanjkseppan commentedAug 17, 2021
edited
Loading

On Linux and Mac. Add a font_manager test that loads the font and uses it. This could be useful in testing the multi-font support (#20740 etc).

I couldn't get the font installed on Windows. There is a Chocolatey installer that installs all of the Noto fonts, which takes a really long time to run.

PR Summary

PR Checklist

  • Has pytest style unit tests (andpytest passes).
  • IsFlake 8 compliant (runflake8 on changed files to check).
  • New features are documented, with examples if plot related.
  • Documentation is sphinx and numpydoc compliant (the docs shouldbuild without error).
  • Conforms to Matplotlib style conventions (installflake8-docstrings and runflake8 --docstring-convention=all).
  • New features have an entry indoc/users/next_whats_new/ (follow instructions in README.rst there).
  • API changes documented indoc/api/next_api_changes/ (follow instructions in README.rst there).

@jkseppanjkseppanforce-pushed thewqy-zenhei-install branch 7 times, most recently from66965e0 tofd2d801CompareAugust 18, 2021 15:56
@QuLogic
Copy link
Member

Zen Hei seemspackaged for Ubuntu, but in Universe, Not sure if we need to do something special to allow installing it.

@jkseppanjkseppan changed the titleTry to install WenQuanYi Zen Hei fontTry to install Google Noto Sans fontsAug 19, 2021
@jkseppan
Copy link
MemberAuthor

The SourceForge download of Zen Hei sometimes fails, so this was probably not a good idea. Google's Noto fonts seem to be better packaged in various systems, so perhaps something like Noto Sans CJK SC could be more useful.

For context: some of the new font changes by@aitikgupta crash on my computer (an M1 Mac) but not on his (I believe Linux on x86), so I would like to add a more cross-platform test with a CJK font and see if we can replicate the crash in CI.

@jkseppanjkseppanforce-pushed thewqy-zenhei-install branch 2 times, most recently fromc04d923 toa71185cCompareAugust 20, 2021 08:31
On Linux and Mac. Add a font_manager test that loads the font anduses it. This could be useful in testing the multi-font support(matplotlib#20740 etc).I couldn't get the font installed on Windows. There is a Chocolateyinstaller that installs all of the Noto fonts, which takes a reallylong time to run.
@jkseppan
Copy link
MemberAuthor

I'm giving up on Windows - the Chocolatey package installs all of the Noto fonts and takes half an hour to do it, and my PowerShell skills are not enough to install just one.

@jkseppanjkseppan changed the titleTry to install Google Noto Sans fontsTry to install the Noto Sans CJK fontAug 20, 2021
@jkseppanjkseppan marked this pull request as ready for reviewAugust 20, 2021 10:17
Copy link
Contributor

@aitikguptaaitikgupta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

I think this can be a good validation for testing the font fallback.

If we do keep this, do we also need to keep installing ZenHei for CI (on Linux) as well?
Noto CJK could potentially replace it right? (I'll do a local installation to test out the actual difference between number of glyphs)

@anntzeranntzer merged commit47077d1 intomatplotlib:masterAug 30, 2021
@QuLogicQuLogic added this to thev3.6.0 milestoneAug 31, 2021
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@anntzeranntzeranntzer approved these changes

@aitikguptaaitikguptaaitikgupta approved these changes

Assignees
No one assigned
Projects
None yet
Milestone
v3.6.0
Development

Successfully merging this pull request may close these issues.

4 participants
@jkseppan@QuLogic@anntzer@aitikgupta

[8]ページ先頭

©2009-2025 Movatter.jp