ComputerScienceHouse / ComputerScienceHouse/packet
Cache RIT Images better
- Ngôn ngữ chính
- Python
- Star
- 25
- Fork
- 33
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
This is a bad cache
https://github.com/ComputerScienceHouse/packet/blob/f61cd2b5b616e3b7f35fad95b515313ea2d50bcb/packet/context_processors.py#L58-L70
It caches the first call to this function for any given username (up to 256 of them) via [`functools.lru_cache`](https://docs.python.org/3/library/functools.html#functools.lru_cache), which is bad, because it kinda defeats the point of gravatar.
We _could_ do some form of caching, but probably shouldn't. Loading these images lazily in the frontend would probably be much better.
At the very least, please remove the cache so we can stop kicking the pods to have photos update.
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Hướng nghiên cứu
Start in packet/context_processors.py at lines 58-70 and inspect the functools.lru_cache usage around the RIT image lookup. Remove the cache as the minimum fix, then verify that updated images can appear without restarting the pods; lazy frontend loading is a possible alternative mentioned by the issue but is not required.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- python
- Lĩnh vực
- backend, performance
- Loại issue
- Lỗi
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 55/100