search

flipoyo / MOLONARI1D

Eclipse Public License 2.0
1 stars 2 forks source link

I don't understand the merges of branch 2023lambert3 #12

Open flipoyo opened 5 days ago

flipoyo commented 5 days ago

est-ce que l'on pourrait en discuter car je pense qu'une partie des bugs SQL viennent de cette branche qui avait était mergée de manière pirate via Thune (avec des copier coller dans ces sources).

fabien-ors commented 5 days ago

La structure de la base de données dans la branche main a très peu évolué en 2023 : https://github.com/flipoyo/MOLONARI1D/commits/main/Molonaviz/src/molonaviz/docs/ERD_structure.sql

Les modifications de la structure de la BDD présentes dans la branche 2023lambert3 depuis le 6 novembre https://github.com/flipoyo/MOLONARI1D/commits/2023lambert3/Molonaviz/src/molonaviz/docs/ERD_structure.sql (création de la table InputMCMC par exemple) ... n'ont jamais été mergées.

Le code proposé dans SPointCoordinator.py (méthode get_table_model par exemple): https://github.com/flipoyo/MOLONARI1D/compare/main...2023lambert3#diff-0fdaa0151bdd614617f1b165326e1318ac6132ba30e733252275115f881541ab est visiblement obsolète quand on voit la tête de la même méthode dans la branche main: https://github.com/flipoyo/MOLONARI1D/blob/main/Molonaviz/src/molonaviz/backend/SPointCoordinator.py

Est-on certain de vouloir garder cette branche?