search

dataforgoodfr / 12_bloom

23 stars 11 forks source link

[BACK] Faire un script SQL pour mettre a jour la table fct_excursion #233

Closed SebM42 closed 2 days ago

SebM42 commented 4 days ago

2 maj à faire :

Rappel : _total_time_defaultais = somme des durées des segments de type DEFAULT_AIS appartenant à cette excursion

Ce script est à passer juste une fois, le calcul étant automatisé pour les excursions futures.

njouanin commented 4 days ago

Il faut aussi mettre à jour le traitement create_update_excursions_segments.py pour le flux. La modification consiste à supprimer le calcul de total_time_extincting_amp et à la remplacer par le calcul de total_time_default_ais comme étant la somme des segments de type DEFAULT_AIS

SebM42 commented 4 days ago

je croyais que tu avais déjà fais le changement concernant le flux (cf #221 )

njouanin commented 4 days ago

Oui j'ai fais le changement pour total_time_extincting_amp . Là je comprends qu'il faut remplacer cette colonne total_time_default_ais dont la règle de calcul change légèrement.

SebM42 commented 4 days ago

ah ? qu est ce qui change entre ce que j ai dit ici et en #221 ? je pensais avoir dit la meme chose, c'est juste que pour cette issue il faut renommer la colonne et faire le calcul rétroactivement avec un script

njouanin commented 4 days ago

tel que je l'ai compris: total_time_extincting_amp=somme du temps passé en DEFAULT_AIS dans une AMP (puisqu'il y a amp dans le nom de la colonne j'en ai déduis qu'il fallait compter le temps en defaut AIS uniquement lorsque le navire se trouve dans une AMP) total_time_default_ais=somme du temps passé en DEFAULT_AIS peu importe où

SebM42 commented 4 days ago

ah ok, en effet je me suis rendu compte un peu tard que le nom de colonne ne correspondait plus, est ce que tu peux t'occuper de modifier le code dans la task stp ? @njouanin

njouanin commented 2 days ago

Requête de MAJ des temps total_time_default_ais pour toutes les excursions:

update fct_excursion fe set total_time_default_ais = (
select sum(segment_duration) from fct_segment fs2 
where "type" = 'DEFAULT_AIS' and excursion_id = fe.id 
)
njouanin commented 2 days ago

Le traitement create_update_excursions_segments.py a été modifié L'historique des temps passés en DEFAUT_AIS dans chaque excursion a été recalculé sur l'ensemble des excursions en BDD.