Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lenteurs du site #461

Open
3 of 6 tasks
IGNF-Xavier opened this issue Jul 2, 2024 · 9 comments
Open
3 of 6 tasks

Lenteurs du site #461

IGNF-Xavier opened this issue Jul 2, 2024 · 9 comments
Labels
bug Anomalie d'une fonctionnalité existante

Comments

@IGNF-Xavier
Copy link

IGNF-Xavier commented Jul 2, 2024

  • Investiguer les lenteurs du site
  • faire remonter, si problème détecté à ce niveau, les causes à
    • Socle (évolution IGNGPF-2799 à utiliser côté client)
    • Architecture IGN-Mut (mis hors de cause)
  • optimisations côté client
@IGNF-Xavier IGNF-Xavier added the bug Anomalie d'une fonctionnalité existante label Jul 2, 2024
@IGNF-Xavier IGNF-Xavier added this to the MVP-0.2_September2024 milestone Jul 2, 2024
@IGNF-Xavier
Copy link
Author

IGNGPF-2799 pourrait contribuer à l'amélioration des temps de réponse

@cedric-esnault-ign
Copy link

Bonjour Xavier,
peux-tu décrire ici des cas reproductibles de lenteurs constatés ?
Merci

@IGNF-Xavier
Copy link
Author

Bonjour @cedric-esnault-ign
Il suffit de se connecter et d'aller dans son espace perso sur le site pour le constater.
A priori l'évolution 2799 devrait permettre de diminuer le nombre de requêtes vers l'entrepôt pour assurer un affichage sur le site.

@slafayIGN
Copy link
Contributor

slafayIGN commented Sep 2, 2024

La possibilité de demander davantage d'informations dès le GET sur des collections à l'API pourrait permettre de refactoriser (en leur évitant de boucler et d'appeler chacun des objets de la collection pour avoir les détails) :

  • UserApiService:getMyDatastores() pourrait accélérer l'affichage du tableau de bord (voir aussi Chargement par morceau du tableau de bord #521)
  • StoredDataApiService:getAllDetailed() pourrait ne plus être utilisée et accélérer la récupération de la liste des fiches de données (tout comme d'autres fonctions getAllDetailed() implémentées pour d'autres entités (uploads, metadata, configurations)

Edit Arnest :

  • la publication d'une métadonnées pourrait être améliorée parce qu'actuellement il y a une boucle sur les configurations pour construire la liste des flux (dans CartesMetadataApiService:getMetadataLayers)
  • la page de fiche de données aussi peut être améliorée parce qu'il y a plusieurs getAllDetailed
  • ❗ il faut aussi faire attention aux effets de bords côté front, parce que les types (typescripts) ne seront plus bons

@slafayIGN slafayIGN added the blocked Nécessite une évolution d'API, de config d'environnement, ou la fourniture de contenu label Nov 6, 2024
@slafayIGN
Copy link
Contributor

Ajout du tag blocked car IGNGPF-2799 ne semble pas avoir bougé depuis le mois de septembre.

Par ailleurs un petit changement du tableau de bord n'accélère pas les choses mais améliore un peu l'expérience utilisateur en affichant un squelette à la place des espaces de travail pendant qu'ils sont en cours de chargement : 9ed72dc

@slafayIGN slafayIGN removed the blocked Nécessite une évolution d'API, de config d'environnement, ou la fourniture de contenu label Nov 29, 2024
@slafayIGN
Copy link
Contributor

IGNGPF-2799 est en prod depuis le 25/11/2024

@slafayIGN slafayIGN removed their assignment Nov 29, 2024
@slafayIGN
Copy link
Contributor

Premières utilisations du champ fields pour diminuer le nombre de requêtes #566

@ocruze
Copy link
Member

ocruze commented Dec 3, 2024

Indications d'améliorations du #566
Dans la page de liste de fiches de données d'un entrepôt :
Nous sommes passés de 3 + nb_uploads + nb_stored_data + (2 * nb_datasheets) requêtes à 3 + (2 * nb_datasheets) requêtes.
Dans l'entrepôt Découverte en prod, qui a 42 fiches de données, nous sommes passés de 225 à 87 requêtes et un temps de chargement de 30-40s à 6-8s.

@ocruze
Copy link
Member

ocruze commented Dec 4, 2024

Bon, j'ai revu la page de liste de données et maintenant nous sommes à 5-10 requêtes pour construire cette page.

ocruze added a commit that referenced this issue Dec 13, 2024
…#577)

* perf: séparation des textes de traduction de leur composant #571 #461

* chore: déplace fichiers traductions proche de leur composant

* chore: convention de nommage .locale pour les fichiers contenant les traductions #571

* fix: corrections suite rebase

* chore: rangement dossiers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Anomalie d'une fonctionnalité existante
Projects
None yet
Development

No branches or pull requests

4 participants