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

feat #204: global endpoints for dashboard rankings #365

Closed
wants to merge 3 commits into from

Conversation

marthevienne
Copy link
Collaborator

@marthevienne marthevienne commented Dec 13, 2024

J'ai enfin compris le ticket #204 et #309 : ils étaient redondants. J'ai donc créé les 2 endpoints globaux pour remplacer ceux pour les classements sur le dashboard :
/metrics/vessels-activity/{category} => liste des navires avec la durée de visite dans les zones de catégorie category dans une plage de temps donnée
/metrics/zones-visited/category} => liste des zones visitées de catégorie category avec la durée de visite dans une plage de temps donnée

@marthevienne marthevienne requested a review from rv2931 December 13, 2024 15:44
@rv2931
Copy link
Collaborator

rv2931 commented Dec 13, 2024

Je pense que pour tout ce qui est durée faudrait commencer a utiliser "duration" plutôt que Time (je l'ai fait aussi ) mais ce sera plus cohérent avec le type et format de valeur retourné

@marthevienne
Copy link
Collaborator Author

Ok, je modifie.

@marthevienne
Copy link
Collaborator Author

Par ailleurs, vu que la table fct_metrics est à peine peuplée, on ne peut pas remonter à plus de 7 jours donc c'est pour ça que les durées restent inchangées quand on modifie la plage de temps

@marthevienne
Copy link
Collaborator Author

Bon il y a des time partout dans l'app donc on changera tout d'un coup par souci de précision :)

@rv2931
Copy link
Collaborator

rv2931 commented Dec 13, 2024

J'ai lancé des rerecalculs des metrics a partir de juin avec le fix de Ewen... J'arrive à 28 juin en une nuit mais sur une plateforme Docker sur Windows les perfos sont un peu minables au bout de quelques heures
Par contre en terme de stratégie de traitement par lot il y a une autre qui serait sûrement plus adaptée et simple : traiter par lot de mmsi du genre "les mmsi < 100000" puis "100000<=mmsi <200000" par exemple (j'ai pas vérifié les échelles de grandeurs. Mais ça permettra d'avoir assez rapidement des données sur toute la période mais que pour certains bateaux

@marthevienne marthevienne changed the title feat #309 migrer endpoint /metrics/activity-in-mpas feat #204: global endpoints for dashboard rankings Dec 13, 2024
@marthevienne
Copy link
Collaborator Author

J'ai fait des erreurs, je la clos pour corriger

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants