cachetools memoizing attributes (cache, key, lock) are not available
まだ誰も着手していません。
- 主要言語
- Python
- スター
- 5.1k
- フォーク
- 2.1k
- 平均マージ
- 1日 19時間
- マージ済み PR(30日)
- 82
説明
According to cachetools’ documentation,
The decorator’s cache, key and lock parameters are also available as
cache,cache_keyandcache_lockattributes of the memoizing wrapper function. […] For the common use case of clearing or invalidating the cache, the decorator also provides acache_clear()[.]
However, this is currently not reflected in the typeshed stubs. cached() and cachedmethod() are declared to return IdentityFunction, i.e. a decorator that takes the function-to-be-cached and returns a callable with identical signature, without the additional properties and functions. It would be great if support for this was added.
Code example
import cachetools
import threading
## cached function
@cachetools.cached(cache=cachetools.LRUCache(maxsize=10),
key=lambda x, y: y,
lock=threading.Lock())
def fun(x: str, y: str) -> str:
return y
print(fun('ignored', 'hello world'))
with fun.cache_lock:
print(fun.cache[fun.cache_key('ignored', 'hello world')])
fun.cache_clear()
## cached method
class Class:
def __init__(self):
self.cache = cachetools.LRUCache(maxsize=10)
self.lock = threading.Lock()
@cachetools.cachedmethod(cache=lambda self: self.cache,
key=lambda self, x, y: y,
lock=lambda self: self.lock)
def meth(self, x: str, y: str) -> str:
return y
c = Class()
print(c.meth('ignored', 'hello world'))
with c.meth.cache_lock(c):
print(c.meth.cache(c)[c.meth.cache_key(c, 'ignored', 'hello world')])
I’m not sure if this is the “intended” way to use the memoizing attributes of a cached method (passing in c for the self argument explicitly), but it seems to work at runtime, at least. (Personally, I’d be fine with support just for @cached functions, but I figured it makes sense to include @cachedmethod in the example too.)
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
まず cachetools.cached() と cachedmethod() の typeshed スタブを見つけ、現在の戻り値の型を cachetools に記載されているメモ化属性と比較します。適切な箇所に cache、cache_key、cache_lock、cache_clear の型付けサポートを追加し、提供されている cached-function と cached-method の例を関連する型チェックテストで検証します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- python
- 領域
- developer-experience
- issue の種類
- バグ
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 42/100