Feat/cache #400
Feat/cache #400
Conversation
atlas/static/index.js
Outdated
| @@ -0,0 +1,26 @@ | |||
| if(configuration.AFFICHAGE_STAT_GLOBALES) { | |||
There was a problem hiding this comment.
Ajouter un espace entre "if" et "(" pour que cela soit plus lisible.
Idem sur le 2ème "if".
docs/installation.rst
Outdated
| :: | ||
|
|
||
| source venv/bin/activate | ||
| pip install -r requirements-dev.txt |
There was a problem hiding this comment.
Il n'y a pas de fichier requirements-dev.txt. Il faudrait soit le créer (si c'est nécessaire) soit supprimer cette section.
docs/installation.rst
Outdated
| :: | ||
|
|
||
| source venv/bin/activate | ||
| flask run |
There was a problem hiding this comment.
Avant, il faut ajouter: cp .flaskenv.sample .flaskenv
There was a problem hiding this comment.
Ne semble pas fonctionner en l'état depuis la racine du dépôt.
Voir le problème indiqué dans le commentaire ci-dessus.
atlas/app.py
Outdated
| if __name__ == "__main__": | ||
| # validation de la configuration secrète | ||
| app = create_app() | ||
| app.run( | ||
| host="0.0.0.0", | ||
| port=secret_conf["GUNICORN_PORT"], | ||
| debug=secret_conf["modeDebug"], | ||
| ) |
There was a problem hiding this comment.
Je n'ai pas réussi à lancer l'appli avec flask run. J'ai une erreur au lancement:
flask.cli.NoAppException: While importing "atlas.atlas.app", an ImportError was raised:
Traceback (most recent call last):
File "/home/jpmilcent/workspace/cbna/sinp/atlas/venv/lib/python3.7/site-packages/flask/cli.py", line 240, in locate_app
__import__(module_name)
File "/home/jpmilcent/workspace/cbna/sinp/atlas/atlas/app.py", line 8, in <module>
from atlas.env import config, secret_conf, cache
ModuleNotFoundError: No module named 'atlas.env'Apparemment, avec ce mécanisme de lancement il faut changer les chemins d'acccès au fichier dans l'appli :
from atlas.env import config, secret_conf, cache
devient
from .env import config, secret_conf, cache
En attendant, de voie comment régler ce problème, j'ai pu lancer l'appli avec python atlas/app.py après avoir ajouter ceci à la fin de app.py:
if __name__ == "__main__":
app = create_app()
app.run()There was a problem hiding this comment.
Est-ce que tu as lancé l'application depuis le repertoire 'atlas' ?
There was a problem hiding this comment.
Est-ce que tu as lancé l'application depuis le repertoire 'atlas' ?
J'ai essayé aussi sans succès. Dès que je lance http://127.0.0.1:8081/ dans mon navigateur, j'ai l'erreur.
A noter que le .flaskenv est à la racine.
There was a problem hiding this comment.
Le lancement de la commande pip install -e . dans le dossier contenant le fichier setup.py ne résout pas le problème.
| @cache.cached() | ||
| def main_stat(): | ||
| connection = utils.engine.connect() | ||
| return vmObservationsRepository.statIndex(connection) |
There was a problem hiding this comment.
Je suis assez surpris par la rapidité d’exécution des requêtes présentes dans statIndex(). Le passage à la v1.5 et le changement dans les VM en est peut être à l'origine.
En tout cas, le cache semble remplir son rôle sur une base avec 11 millions d'observations. En relançant, l'appli le cache est bien vidé.
atlas/app.py
Outdated
| if __name__ == "__main__": | ||
| # validation de la configuration secrète | ||
| app = create_app() | ||
| app.run( | ||
| host="0.0.0.0", | ||
| port=secret_conf["GUNICORN_PORT"], | ||
| debug=secret_conf["modeDebug"], | ||
| ) |
There was a problem hiding this comment.
Est-ce que tu as lancé l'application depuis le repertoire 'atlas' ?
J'ai essayé aussi sans succès. Dès que je lance http://127.0.0.1:8081/ dans mon navigateur, j'ai l'erreur.
A noter que le .flaskenv est à la racine.
| # Temps en seconde de la durée du cache | ||
| # utilisé pour les statistiques de la page d'accueil (stats global et rang taxonomique) | ||
| # à synchroniser avec le raffraichissement des VM | ||
| CACHE_TIMEOUT = fields.Number(load_default=3600) |
There was a problem hiding this comment.
J'ai du seulement mettre CACHE_TIMEOUT = 3600 ici.
There was a problem hiding this comment.
Tu as fait un pip install -e . depuis la racine ?
| @@ -26,6 +26,8 @@ | |||
| $(".lazy").lazy(); | |||
| }); | |||
| </script> | |||
| <script src="{{ url_for('static', filename='index.js') }}"></script> | |||
There was a problem hiding this comment.
Il faudrait rajouter cette ligne dans un test qui vérifie que l'on affiche bien les stats globale sur la page d'accueil : {% if configuration.AFFICHAGE_STAT_GLOBALES or configuration.AFFICHAGE_RANG_STAT %}.
Sinon, on charge un fichier pour rien...
There was a problem hiding this comment.
Il y a déjà des IF dans le fichier js. Et si un jour on ajoute encore d'autre truc dans ce index.js, qui eux ne sont pas dépendant de paramètres ..
docs/installation.rst
Outdated
| :: | ||
|
|
||
| source venv/bin/activate | ||
| flask run |
There was a problem hiding this comment.
Ne semble pas fonctionner en l'état depuis la racine du dépôt.
Voir le problème indiqué dans le commentaire ci-dessus.
Il faudrait passer ces routes en API, et mettre un cache spécifique dessus. J'ai pas le temps de le faire là. Je propose qu'on merge ça en l'état pour éviter que la PR diverge. On pourra le faire par la suite en suivant le même exemple |
@TheoLechemia Ok, ça me va. Par contre, il faut résoudre le problème du |
|
J'ai relancé un |
243a8f2 to
a02e521
Compare
a02e521 to
3f71b99
Compare
Utilisation de la lib flask-cache pour le calcul des statistiques de la page d'acceuil.
Les stats sont passés en API