-
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
fix getVesselsInActivity #287
Conversation
On a besoin des deux, |
J'ai fait un trackedCount, faut effectivement faire un activityCount sur le même modèle avec paramètres de dates |
Je m'occupe du activityCount. |
94b84a3
to
1174267
Compare
j'ai rebase. il y avait un conflit |
…al time spent at sea (correct sum) # Conflicts: # backend/bloom/services/metrics.py
1174267
to
552276e
Compare
L'endpoint va évolué du coup, je sais pas trop. On va éviter de casser le front donc on peut laisser tel quel et créer le bon endpoint en suivant, migrer le front vers ce nouveau endpoint puis supprimer getVesselsInActivity ? |
@rv2931 et @alexphiev j'ai créé un nouveau endpoint qui a un nom plus accurate pour le nombre de bateaux en mer pendant une période donnée : |
J'ai corrigé la méthode getVesselsInActivity (on comptait les excursions, pas les navires). Je me retrouve avec un nombre non aberrant de navires en activité pendant une période donnée (1464 au max).
@alexphiev @rv2931 @SebM42
-> Est-ce que le résultat de cette méthode va nous servir ? ou faut-il remplacer la méthode par une méthode qui renvoie juste le nombre de navires actifs comme on en a parlé hier en réunion ?