Skip to content

[MRG+2] MAINT: Refactor the converted-image cache #7764

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.

Sign up for GitHub

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

Closed
wants to merge 19 commits into from
Closed

[MRG+2] MAINT: Refactor the converted-image cache #7764

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
1b24a16
Factor the converted-image cache out of compare.py
jkseppan Jan 7, 2017
650ddf2
Import cbook again
jkseppan Jan 21, 2017
3b5f636
Lock the cache directory
jkseppan Jan 21, 2017
556fd82
Communicate node-specific cache reports to master if using xdist
jkseppan Jan 22, 2017
9f5a06d
Delete the nose plugin
jkseppan Jan 25, 2017
33f2528
Reinstate deleted functions
jkseppan Jan 25, 2017
2136a26
Make conversion_cache private
jkseppan Jan 25, 2017
6995a27
Make globals private
jkseppan Jan 25, 2017
9ea66dd
Improve wording
jkseppan Jan 25, 2017
3751f01
mkdirs throws OSError
jkseppan Jan 25, 2017
a70081e
Improve documentation in response to review
jkseppan Jan 30, 2017
85f54de
Use pytest facilities
jkseppan Jan 30, 2017
0d51e03
Simplify gs and inkscape version checking logic
jkseppan Jan 30, 2017
9de06c0
pep8 fixes
jkseppan Jan 30, 2017
6007b37
There is no nose plugin any more
jkseppan Feb 18, 2017
e52cdd2
Import the pytest hooks in the other conftest.py files
jkseppan Feb 20, 2017
e5184bb
Omit unnecessary tmpdir deletions
jkseppan Feb 22, 2017
f47cbe4
Move pytest_addoption call to top-level conftest.py
jkseppan Feb 22, 2017
996f22b
pep8 fix
jkseppan Feb 22, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Improve wording
  • Loading branch information
@jkseppan
jkseppan committed Feb 20, 2017
commit 9ea66ddf0f97ce30800c6c7081f2ac7d23e14f28
9 changes: 6 additions & 3 deletions lib/matplotlib/testing/_conversion_cache.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ class _ConversionCache(object):
Parameters
----------
directory : str, optional
Files are stored in this directory, defaults to `'test_cache'` in
Files are stored in this directory. Defaults to `'test_cache'` in
the overall Matplotlib cache directory.
max_size : int, optional
The flush method will delete files until their combined size is
Expand All @@ -54,7 +54,8 @@ def __init__(self, directory=None, max_size=int(1e8)):
self.cachedir = self.get_cache_dir()
self.ensure_cache_dir()
if not isinstance(max_size, int):
raise ValueError("max_size is %s, expected int" % type(max_size))
raise ValueError("max_size is of type %s, expected int" %
type(max_size))
self.max_size = max_size
self.cached_ext = '.png'
self.converter_version = {}
Expand Down Expand Up @@ -120,7 +121,9 @@ def _get_file_hash(self, path, block_size=2 ** 20):
if version_tag is None:
warnings.warn(
("Don't know the external converter for files with extension "
"%s, cannot ensure cache invalidation on version update.")
"%s, cannot ensure cache invalidation on version update. "
"Either the relevant conversion program is missing or caching"
" is attempted for an unknown file type.")
% ext)
result = self._get_file_hash_static(path, block_size, version_tag)
self.hash_cache[path] = result
Expand Down