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

Résolution du problème des alertes qui remontent malgré la présence d'une suspension d'alerte #3971

Merged
merged 9 commits into from
Dec 18, 2024

Conversation

VincentAntoine
Copy link
Collaborator

@VincentAntoine VincentAntoine added bug Something isn't working data labels Dec 13, 2024
@VincentAntoine VincentAntoine self-assigned this Dec 13, 2024
@VincentAntoine VincentAntoine force-pushed the vincent/fix_silenced_alerts branch from be3472f to 09d40df Compare December 13, 2024 16:13
@VincentAntoine VincentAntoine marked this pull request as ready for review December 17, 2024 08:51
@VincentAntoine
Copy link
Collaborator Author

@louptheron dans l'idée, jusqu'ici le filtrage des silenced_alerts était basé sur l'heure d'exécution du flow : si un navire avait une silenced_alert jusqu'à 13:30, alors les runs de flows ne pouvaient pas remonter d'alerte sur ce navire jusqu'à 13:30, mais à partir de 13:30 une alerte pouvait remonter sur ce navire pour signaler une suspicion d'infraction (pêche, FAR manquant) détectée sur un élément déclencheur (piste VMS, données déclaratives) datant d'avant 13:30. Donc typiquement un chalutage 3 nautiques détecté sur la piste VMS à 13h30, acquitté par un superviseur à 13h35, pouvait remonter à nouveau à 13h45 pour le même tronçon de piste VMS de 13h30, pourtant déjà acquitté.
Pour corriger ceci, j'introduis la notion de triggering_behaviour_datetime_utc qui décrit la date de l'élément déclenchant (position VMS, message JPE) et c'est à cette date que s'applique les silenced_alerts. Ainsi un tronçon VMS acquitté ne remontera pas une nouvelle fois, il faudra une nouvelle position VMS ultérieure à celle acquittée pour que ça puisse à nouveau remonter sur un même navire.

@louptheron louptheron merged commit 95302c9 into master Dec 18, 2024
8 checks passed
@louptheron louptheron deleted the vincent/fix_silenced_alerts branch December 18, 2024 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working data
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Les suspensions d'alertes ne fonctionnent pas correctement
2 participants