Repository navigation
Deprecate/remove maxdict #22278
Description
The maxdict in matplotlib's cbook is only used in one place in Matplotlib.
matplotlib/lib/matplotlib/text.py
Line 111 in bedc202
It can probably be updated to functools.cache now, with some work to figure out exactly where to put the cache decorator.
@greglucas Actually, switching to lru_cache would be a bit annoying, because the caching on fontproperties must be based on hash(fontproperties) (as fontproperties themselves are mutable) but then you can't get back the fontproperties from the hash...
(Again, it may be possible to add more indirection around that but it may not really be worth it for now...)
Originally posted by @anntzer in #22271 (comment)
Activity
timhoffm commented on Jan 21, 2022
We can immediately deprecate it, and still have enough time to figure out whether we need it internally or can remove it.
anntzer commented on Jan 28, 2022
Done by #22271.