Ricordisamoa has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/261803

Change subject: functools.ttl_cache() does not exist, always use cachetools
......................................................................

functools.ttl_cache() does not exist, always use cachetools

I got it confused with functools.lru_cache()

Change-Id: I7b8ab2f2124fa133dcb0ed5b68a6cfb5b1bf9123
---
M chemistry.py
1 file changed, 1 insertion(+), 5 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/labs/tools/ptable 
refs/changes/03/261803/1

diff --git a/chemistry.py b/chemistry.py
index 216f7aa..5a05c90 100644
--- a/chemistry.py
+++ b/chemistry.py
@@ -24,15 +24,11 @@
 from urllib.request import urlopen
 from collections import defaultdict
 
-try:
-    from functools import ttl_cache
-except ImportError:
-    from cachetools import ttl_cache
+from cachetools import ttl_cache
 
 import data
 
 
-# cachetools does not support maxsize=None
 @ttl_cache(maxsize=20, ttl=21600)
 def get_json_cached(url, data):
     """The information is cached for 6 hours."""

-- 
To view, visit https://gerrit.wikimedia.org/r/261803
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7b8ab2f2124fa133dcb0ed5b68a6cfb5b1bf9123
Gerrit-PatchSet: 1
Gerrit-Project: labs/tools/ptable
Gerrit-Branch: master
Gerrit-Owner: Ricordisamoa <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to