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

[DASHBOARD] Erreur dans le calcul et données dynamiques à envoyer au front #245

Closed
SebM42 opened this issue Nov 8, 2024 · 7 comments
Closed
Assignees
Labels

Comments

@SebM42
Copy link
Collaborator

SebM42 commented Nov 8, 2024

image

Total vessel in activity :

Numérateur : Corriger le calcul
Rappel : représente le nombre de navires uniques et actifs qui ont participé à au moins 1 excursion dans la plage de temps donnée
--> ajouter un endpoint pour avoir les valeurs (juste un chiffre)

--> pour l'instant on garde le dénominateur fixe.

Total amp visited :

Dénominateur : Valeur dynamique à envoyer au front
Rappel : représente le nombre total de zones de catégories amp dans la base
--> ajouter un end point pour avoir les valeurs. (juste un chiffre)

@rv2931
Copy link
Collaborator

rv2931 commented Nov 15, 2024

Je viens de regarder le code, la valeur 10000 vient d'une valeur par défaut quand il n'y avait pas les endpoints
J'ai l'impression que ce n'est plus le cas, ça ne devrait donc plus être 10000 maintenant
par contre pour les devs qui utilisent la stack Docker, il faut rebuild le conteneur frontend pour toutes les mises à jour frontend
J'ai de recompiler le frontend de mon côté voir si c'est toujours 10000
D'ailleurs c'est un peu bizarre 10000 comme valeur par défaut, je proposerais soit un ? ou - ou éventuellement -1 si ça doit être un entier mais pas 10000

@louwelgryn
Copy link
Collaborator

assez d'accord que ca serait bien une valeur par défaut qui nous laisse comprendre que ca n'est pas de la vrai donnée.
? c'est pas mal ;)

@louwelgryn
Copy link
Collaborator

@marthevienne vérifier la donnée sous jacente. vessel_in_activity

@marthevienne
Copy link
Collaborator

Done !

Note : j'ai corrigé le code dans la PR #287 mais la méthode en soit n'est pas utile comme on en a parlé au weekly. On récupère en sortie la liste des navires qui ont fait au moins une excursion pendant une plage de temps donnée avec le temps en mer associé. On n'en a pas besoin donc supprimer ce endpoint en le détournant vers l'issue existante pour retourner un nombre, pas une liste.

@marthevienne
Copy link
Collaborator

Hello @rv2931, qui s'occupe des endpoints ?

@rv2931
Copy link
Collaborator

rv2931 commented Nov 26, 2024

Je suis en train de regarder : #298
Là a partir d'un dump de cette semaine, je ne trouve des zones visitées qu'au mois de novembre (44)
Faut que je regarde les données d'un peut plus près pour voir si c'est cohérent ou non

@marthevienne
Copy link
Collaborator

Hello @rv2931, je m'occupe de l'endpoint activityCount sur le modèle de ce que tu as fait pour trackedCount

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