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

feat: clean tables and rename table/schema #151

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

rv2931
Copy link
Collaborator

@rv2931 rv2931 commented Apr 5, 2024

Je suis en train de regarder pour faire un petit ménage dans a base
Quelques questions me viennent:

  • Qu'est ce que les White Zone ? ou Zone blanche: ça apparait dans le Miro issues des vessel_positions mais pas définie dans le Glossaire : https://dataforgood.notion.site/6a8d74632b0c45ebb0990c2e75147f57?v=a61dbe31b01a47149974e3bb5e9a3c16

  • la table mpa_fr_with_mn n'est plus utilisée on est d'accord ? un fichier CSV amp les charge directement dans dim_zone c'est bien ça ?

  • la table spire_vessel_positions n'est plus utilisée on est d'accord ? un fichier CSV spire_ais_data les charge directement dans la table stg_spire_ais_data c'est bien ça ?

  • spatial_ref_sys : il me semble que c'est une table postgis mais je suis pas complètement certain

  • Quid de vessel_data, vessel_positions, vessel_voyages et vessels, alert ?

  • @nicolas Jouanin: quelles sont les autres tables issues des données API Spire qu'on devrait stocker considérer comme du "staging" ? (même si pas utilisées pour le moment) actuellement j'ai identifié stg_spire_ais_data (staging => table tampon entre une source et son utilisation dimensions/faits) mais tu disais qu'on pouvait déjà récupérer d'autres infos de spire

@rv2931 rv2931 force-pushed the clean_tables_remove_and_rename branch 2 times, most recently from 2450c8c to b725880 Compare April 5, 2024 15:49
@njouanin
Copy link
Collaborator

njouanin commented Apr 5, 2024

Dans le fichier src/bloom/infra/database/sql_model.py, il faut également modifier le tablename de la classe SpireAisData

@rv2931 rv2931 force-pushed the clean_tables_remove_and_rename branch from e5a09c3 to 81277b1 Compare April 7, 2024 11:53
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.

2 participants