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

fix getVesselsInActivity #287

Merged
merged 3 commits into from
Nov 29, 2024
Merged

fix getVesselsInActivity #287

merged 3 commits into from
Nov 29, 2024

Conversation

marthevienne
Copy link
Collaborator

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 ?

@alexphiev
Copy link
Collaborator

On a besoin des deux, getVesselsInActivity pour le top 10 dans le dashboard et une méthode pour le total. Idéalement il faudrait en effet une méthode qui ne renvoie que le total pour que je n'aie pas à récupérer des données inutiles.

@rv2931
Copy link
Collaborator

rv2931 commented Nov 29, 2024

J'ai fait un trackedCount, faut effectivement faire un activityCount sur le même modèle avec paramètres de dates

@marthevienne
Copy link
Collaborator Author

marthevienne commented Nov 29, 2024

Je m'occupe du activityCount.
@alexphiev pour le classement top vessels, le endpoint ne renvoie pas les bonnes infos comme mentionné ici #309. En revanche, on garde le format du JSON, le nom changera sûrement.

@rv2931 rv2931 force-pushed the fix/number_active_vessels branch from 94b84a3 to 1174267 Compare November 29, 2024 13:40
@rv2931
Copy link
Collaborator

rv2931 commented Nov 29, 2024

j'ai rebase. il y avait un conflit
faut l'intégrer du coup ?

…al time spent at sea (correct sum)

# Conflicts:
#	backend/bloom/services/metrics.py
@rv2931 rv2931 force-pushed the fix/number_active_vessels branch from 1174267 to 552276e Compare November 29, 2024 13:45
@marthevienne
Copy link
Collaborator Author

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 ?

@marthevienne
Copy link
Collaborator Author

@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 :
/vessels/vessels-at-sea
@rv2931 tu peux review la PR ?

@marthevienne marthevienne merged commit 138cc3a into main Nov 29, 2024
0 of 7 checks passed
@rv2931 rv2931 deleted the fix/number_active_vessels branch November 30, 2024 12:45
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.

3 participants