search

PnX-SI / GeoNature-atlas

Application WEB permettant de générer des fiches espèces publiques à partir d'observations faune/flore
GNU General Public License v3.0
44 stars 46 forks source link

Fix: add indexes to vm_observations_mailles #393

Closed jpm-cbna closed 1 month ago

jpm-cbna commented 2 years ago

Improve atlas.vm_observations_mailles with new indexes.

camillemonchicourt commented 2 years ago

A mettre dans un petit fichier d'update SQL aussi je pense pour ceux qui auraient déjà la 1.5.0 installée.

jpm-cbna commented 2 years ago

A mettre dans un petit fichier d'update SQL aussi je pense pour ceux qui auraient déjà la 1.5.0 installée.

Oui, bien vu ! Mais par contre, je voie plusieurs solutions pour le faire :

Concernant le fichier data/update/update_vm_observations.sql, il me semble qu'il devrait être remplacé par un script Bash. Cela permettrait de rejouer les scripts présents dans le dossier data/atlas/ pour éviter de dupliquer le code SQL. Il pourrait aussi se servir correctement des variables définies dans settings.ini plutôt que d'avoir l'utilisateur geonatatlas en dur dans le code.

camillemonchicourt commented 2 years ago

En effet quand on met à jour GeoNature-atlas, si possible on ne demande pas de récréer toutes les VM car souvent chacun les a adapté à son contexte. Donc une fichier SQL avec les évolution depuis la dernière version est idéal.

On avait déjà évoqué le fait que garder en doublon partiel ce fichier data/update/update_vm_observations.sql était pas une bonne idée, pas à jour car en partie en doublon, donc à dégager selon moi (https://github.com/PnX-SI/GeoNature-atlas/issues/346#issuecomment-908141339).

jpm-cbna commented 2 years ago

@TheoLechemia Je viens rajouter le fichier "update_vm_index.sql" dans le dossier data/update/. Il pourra servir à constituer le fichier de migration update_1.5.1to1.5.2.sql.

jpm-cbna commented 1 month ago

Les changements réalisés récemment sur la VM vm_observations_mailles et intégré à la branche develop rende cette PR inutile.