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

[API] Migrer endpoint /metrics/activity-in-mpas #309

Open
marthevienne opened this issue Nov 29, 2024 · 4 comments
Open

[API] Migrer endpoint /metrics/activity-in-mpas #309

marthevienne opened this issue Nov 29, 2024 · 4 comments
Assignees
Labels

Comments

@marthevienne
Copy link
Collaborator

marthevienne commented Nov 29, 2024

Reste à faire : migrer le endpoint sur la table fct_metrics une fois qu'elle sera mise en prod


Le endpoint utilisé actuellement n'est pas le bon.
Il s'agit du endpoint /metrics/vessels-in-activity qui renvoie la liste des navires en activité. La durée affichée est le temps total en mer des excursions qui chevauchent la plage de temps.

Il faut :

  • supprimer ce endpoint qui ne sert à rien (le nombre de navires en activité en mer sera renvoyé par le endpoint /metrics/activityCount /metrics/vessels-at-sea)
  • le remplacer par un endpoint qui renvoie la liste des navires qui ont passé du temps dans des MPAs dans la plage de temps donnée avec la durée passée dans les MPAs dans la plage de temps donnée.

=> @ejamet73 la table fct_metrics pourrait servir pour ce endpoint

@ejamet73
Copy link
Collaborator

ejamet73 commented Nov 29, 2024

pour la table fct_metrics: elle sera parfaite pour ce endpoint, j'attends juste la review d'Hervé.

@rv2931
Copy link
Collaborator

rv2931 commented Nov 29, 2024

En fait le soucis au départ était la définition de de "in activity"
La définition est plus claire si l'on dit que l'activité c'est des segments dans des AMP. Je modifie vite fait le endpoint en attendant de passer sur du fct_metrics

@rv2931
Copy link
Collaborator

rv2931 commented Nov 29, 2024

J'ai modifié la requête actuelle pour retourner la liste des vessels avec le temps cumulé des segments in_mpa=True sur la tranche de date
#313
Requête à migrer sur des données fct_metrics (peut-être en modifiant le nom du endpoints du genre activity-in-mpa..) à terme mais pour le moment la requête est

@rv2931 rv2931 added the api label Dec 8, 2024
@marthevienne marthevienne changed the title [API] Problème endpoint pour classement Top vessels visiting MPAs [API] Migrer endpoint /metrics/activity-in-mpas Dec 10, 2024
@marthevienne marthevienne assigned marthevienne and unassigned rv2931 and alexphiev Dec 11, 2024
@marthevienne
Copy link
Collaborator Author

Je me mets sur cette issue
Bloqué par #337

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

No branches or pull requests

4 participants