-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
check freetype version also specifies what is wrong #6403
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
Changes from 1 commit
5a3a882
977be99
d6456f5
f4de6df
2c5b61e
b8aa778
2400b3f
1f3dc7a
b97e741
eef6b93
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
…ytestring (not writing it to file), which can be practical in combination with the base64-package. Also addded a {figure,pyplot}.subplots_iterator which returns an iterator over subplots, which can be practical in for example ipynb's, where one wants to make many graphs and likes them to be 3 (or 5 or whatever) on a line.
- Loading branch information
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -1513,6 +1513,7 @@ def _jupyter_nbextension_paths(): | |
'matplotlib.tests.test_streamplot', | ||
'matplotlib.tests.test_style', | ||
'matplotlib.tests.test_subplots', | ||
'matplotlib.tests.test_subplots_iterator', | ||
'matplotlib.tests.test_table', | ||
'matplotlib.tests.test_text', | ||
'matplotlib.tests.test_texmanager', | ||
|
@@ -1553,7 +1554,12 @@ def _init_tests(): | |
warnings.warn( | ||
"matplotlib is not built with the correct FreeType version to run " | ||
"tests. Set local_freetype=True in setup.cfg and rebuild. " | ||
"Expect many image comparison failures below.") | ||
"Expect many image comparison failures below. " | ||
"Expected {0} != found {1}".format( | ||
ft2font.__freetype_version__, | ||
LOCAL_FREETYPE_VERSION | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This message might be a little misleading, I would think, because this isn't the full condition that needs to be satisfied. @mdboom, I will defer to you on this. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Although you are right, I've actually set It makes me wonder if There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It does work Travis uses it all the time Please check the Travis logs for instance here There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I am not sure if There is also the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @tacaswell You might be right. What also seemed to have worked is removing the ubuntu, the virtualenv, and re-executing All reactions |
||
) | ||
) | ||
|
||
try: | ||
import nose | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should these be in the other order? We expected
LOCAL_FREETYPE_VERSION
but did not find it.