Cache RIT Images better
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 55/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- python
- Domaine
- backend, performance
Piste de recherche
Commencez dans packet/context_processors.py aux lignes 58-70 et examinez l’utilisation de functools.lru_cache autour de la recherche d’images RIT. Supprimez le cache comme correction minimale, puis vérifiez que les images mises à jour peuvent apparaître sans redémarrer les pods ; lazy frontend loading est une alternative possible mentionnée par l’issue, mais elle n’est pas requise.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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, 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.
- Langage dominant
- Python
- Étoiles
- 25
- Forks
- 33
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.
- Fournit un Dockerfile ou un fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de ComputerScienceHouse/packet
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
ComputerScienceHouse/packet#413 ·
-
Upgrade to bootstrap 5Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 38/100
ComputerScienceHouse/packet#411 ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 30/100
ComputerScienceHouse/packet#408 ·
-
React Frontend?Ouverteux enhancement
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
ComputerScienceHouse/packet#404 ·
-
ux enhancement
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
ComputerScienceHouse/packet#403 ·
Toutes les issues de ComputerScienceHouse/packet
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
vllm-project/vllm#61131 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
kubernetes-client/python#2735 ·
Les mainteneurs répondent en général sous 1 jour
-
desktop help wanted test
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
Broken links found in docsOuvertedocs pydanty:is-working
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
pydantic/pydantic-ai#10113 ·
Les mainteneurs répondent en général sous 1 jour
-
Tutorial unit test leaks sample data into tutorial doctestPeut-être pris @Gambit-Checkmate l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100