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

Renommage de colonnes #237

Closed
SebM42 opened this issue Nov 3, 2024 · 6 comments
Closed

Renommage de colonnes #237

SebM42 opened this issue Nov 3, 2024 · 6 comments
Assignees
Labels

Comments

@SebM42
Copy link
Collaborator

SebM42 commented Nov 3, 2024

Relatif à l issue #224

Dans FCT_EXCURSION renommer les colonnes :

  • total_time_in_costal_waters => total_time_in_zones_with_no_fishing_rights
  • total_time_fishing_in_costal_waters => total_time_fishing_in_zones_with_no_fishing_rights

Dans FCT_SEGMENT renommer la colonne :

  • in_costal_waters => in_zone_with_no_fishing_rights
@njouanin
Copy link
Collaborator

njouanin commented Nov 5, 2024

est-ce que ça change les règles de calcul ? ou bien c'est purement un renommage de forme.

@SebM42
Copy link
Collaborator Author

SebM42 commented Nov 5, 2024

A ma connaissance, total_time_in_costal_waters n'est pas calculé à ce jour

Pour ce qui est des modifs de calcul j'ai créé l'issue #238 qui englobent ces 2 calculs à faire

@ejamet73
Copy link
Collaborator

ejamet73 commented Nov 5, 2024

Capture d’écran du 2024-11-05 15-40-49

total_time_in_costal_waters est calculé ici non ?

@SebM42
Copy link
Collaborator Author

SebM42 commented Nov 5, 2024

pardon, je voulais dire in_costal_waters n'est pas calculé (dans la table fct_segment)

dans tous les cas, soit ce n'est pas calculé et il faut le faire, soit c'est calculé mais le calcul change et donc il faut le revoir, et tout est indiqué dans l'issue #238

@njouanin
Copy link
Collaborator

njouanin commented Nov 5, 2024

OK, pour cet issue je ne traite que les renommages de colonnes. Pour les règles de calcul on voit ça dans l'issue #238.

@njouanin
Copy link
Collaborator

njouanin commented Nov 5, 2024

Fait pour le renommage des colonnes demandées
Il y a une migration de base à lancer : alembic upgrade head
La base clevercloud est à jour.

@njouanin njouanin closed this as completed Nov 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants