Warning message prior to fc-list command #5640

Merged
merged 2 commits into from Dec 23, 2015

Conversation

Projects
None yet
8 participants
Contributor

notmatthancock commented Dec 9, 2015

When the font cache is first created, fc-list may take up to several minutes on some machines. No indication is given that this is being done. This adds a warning so the user is aware.

#2919

@notmatthancock notmatthancock Added warning message before fc-list command
5d3f8a7

mdboom added this to the Critical bugfix release (1.5.1) milestone Dec 9, 2015

Owner

mdboom commented Dec 9, 2015

👍

Owner

mdboom commented Dec 9, 2015

Eventually #5414 will fix this altogether, but this is fine for now.

Owner

jenshnielsen commented Dec 9, 2015

👍

I would change the text to something like

"Building Matplotlib font cache by running fc-list. This could take a few minutes."

Member

WeatherGod commented Dec 9, 2015

Maybe "Matplotlib is building {cachename} using fc-list. This may take a moment".

I prefer "moment" over "minutes" because it is ambiguous and thus a bit future-proof. Each person's perception of a moment is shaped by the computer they are using, so it is a bit more flexible, too.

@notmatthancock notmatthancock Added warning message before fc-list command
d275277

@tacaswell tacaswell added a commit that referenced this pull request Dec 23, 2015

@tacaswell tacaswell Merge pull request #5640 from notmatthancock/master
Warning message prior to fc-list command
6e684d1

@tacaswell tacaswell merged commit 6e684d1 into matplotlib:master Dec 23, 2015

3 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.0008%) to 67.782%
Details

tacaswell removed the needs_review label Dec 23, 2015

@tacaswell tacaswell added a commit that referenced this pull request Dec 23, 2015

@tacaswell tacaswell Merge pull request #5640 from notmatthancock/master
Warning message prior to fc-list command
c4a94e5

@tacaswell tacaswell added a commit that referenced this pull request Dec 23, 2015

@tacaswell tacaswell Merge pull request #5640 from notmatthancock/master
Warning message prior to fc-list command
0823022
Owner

tacaswell commented Dec 23, 2015

backported to 2.x as c4a94e5 and v1.5.x as 0823022 (because I did not read the milestone carefully enough).

pelegm commented Jan 11, 2016

One thing I don't understand: why is it giving this warning every time I load matplotlib? Does it build the font cache every time?

Owner

mdboom commented Jan 11, 2016

It should only be rebuilding each time if it doesn't have write permissions to ~/.matplotlib (or ~/.cache/matplotlib on Linux).

pelegm commented Jan 11, 2016

@mdboom, thanks for your quick reply. I am using Ubuntu, my user owns ~/.cache and ~/.cache/matplotlib, and there's indeed fontList.cache file there, but it dates to April last year. Importing matplotlib.pyplot does not seem to touch that file.

Owner

jenshnielsen commented Jan 11, 2016

Do you own that file too and have user write access to it?

pelegm commented Jan 11, 2016

Sure:

peleg@lati::~/.cache/matplotlib$ ls -la
total 308
drwxrwxr-x  3 peleg peleg   4096 Jan 11 18:43 .
drwx------ 50 peleg peleg   4096 Jan 11 11:08 ..
-rw-rw-r--  1 peleg peleg 161444 Apr  6  2015 fontList.cache
-rw-rw-r--  1 peleg peleg 138010 Apr 24  2015 fontList.py3k.cache
drwxrwxr-x  2 peleg peleg   4096 Apr  6  2015 tex.cache

(owner write access...)

Owner

jenshnielsen commented Jan 11, 2016

Which version are you using 1.5.x and 2.x uses a binary font cache but
on master it is changed to using a json fine see #5021

I would suggest moving those files out of the way and try again if you are using 1.5.1 or on the 2.x branch.

pelegm commented Jan 11, 2016

I am using matplotlib 1.5.1, just downloaded via pip.

I have removed the files as you suggested, and after they were created again (during the first time I've imported matplotlib.pyplot), the warning does not appear any more. Thanks for your help!

I run into similar behavior when opening both my system matplotlib of 1.3.1 and my installed version of 1.5.1. Apparently the older file format with matplotlib 1.3.1 always causes matplotlib 1.5.1 to rescan, but not update the file.

I'm glad this warning is here as it means I know to wait for it to run. @pelegm I found that if I deleted everything in ~/.matplotlib (I'm on OSX) it generated once and I've not had the message since.

@joslloand joslloand added a commit to ambisonictoolkit/atk-sc3 that referenced this pull request Jul 15, 2016

@joslloand joslloand Update soundfield transform figures to illustrate rE
commit for the figures for
#11

Will need to follow with an update to SCHelp describing the scales
[gain, r, theta]

NOTE: Fonts and scaling have changed as a result of an issue with
anaconda/matplotlib. Some relevant issues can be found here:

fonnesbeck/ScipySuperpack#39
matplotlib/matplotlib#5836
matplotlib/matplotlib#5640
matplotlib/matplotlib#6721
matplotlib/matplotlib#3590
4f6d2fc

@joslloand joslloand added a commit to supercollider/sc3-plugins that referenced this pull request Jul 16, 2016

@joslloand joslloand ATK: Update Intro-to-the-ATK & imaging figures
More verbose Intro-to-the-ATK. Describe in more detail how to read
transform figures with description of sound field quality measure, rE.

Transform plots now illustrate rE rather than rV, which is more closely
tied to perception.

NOTE: Fonts and scaling have changed as a result of an issue with
anaconda/matplotlib. Some relevant issues can be found here:

fonnesbeck/ScipySuperpack#39
matplotlib/matplotlib#5836
matplotlib/matplotlib#5640
matplotlib/matplotlib#6721
matplotlib/matplotlib#3590
604cea3

@mtmccrea mtmccrea pushed a commit to mtmccrea/migrate-atk-sc-src that referenced this pull request Nov 30, 2016

@joslloand joslloand ATK: Update Intro-to-the-ATK & imaging figures
More verbose Intro-to-the-ATK. Describe in more detail how to read
transform figures with description of sound field quality measure, rE.

Transform plots now illustrate rE rather than rV, which is more closely
tied to perception.

NOTE: Fonts and scaling have changed as a result of an issue with
anaconda/matplotlib. Some relevant issues can be found here:

fonnesbeck/ScipySuperpack#39
matplotlib/matplotlib#5836
matplotlib/matplotlib#5640
matplotlib/matplotlib#6721
matplotlib/matplotlib#3590
864b626
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment