-
Notifications
You must be signed in to change notification settings - Fork 11
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
Comments
Je viens de regarder le code, la valeur 10000 vient d'une valeur par défaut quand il n'y avait pas les endpoints |
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. |
@marthevienne vérifier la donnée sous jacente. vessel_in_activity |
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. |
Hello @rv2931, qui s'occupe des endpoints ? |
Je suis en train de regarder : #298 |
Hello @rv2931, je m'occupe de l'endpoint activityCount sur le modèle de ce que tu as fait pour trackedCount |
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)
The text was updated successfully, but these errors were encountered: