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

[Profil taxon] - Implémenter le SQL #1103

Closed
3 of 4 tasks
TheoLechemia opened this issue Oct 26, 2020 · 5 comments
Closed
3 of 4 tasks

[Profil taxon] - Implémenter le SQL #1103

TheoLechemia opened this issue Oct 26, 2020 · 5 comments

Comments

@TheoLechemia
Copy link
Member

TheoLechemia commented Oct 26, 2020

  • Revoir le code SQL vis à vis de la convention de codage

  • Clarifier / revoir le nom de certaines tables

  • Décrire les champs et les tables

  • Voir le travail de Gil et le fusionner avec l'existant

Adrien-Pajot added a commit to Adrien-Pajot/GeoNature that referenced this issue Oct 28, 2020
@DonovanMaillard
Copy link
Contributor

DonovanMaillard commented Nov 10, 2020

  • Revoir le code SQL vis à vis de la convention de codage
  • Clarifier / revoir le nom de certaines tables
  • Décrire les champs et les tables
  • Voir le travail de Gil et le fusionner avec l'existant
  • Implémenter la notion d'organisme dans le calcul des profiles (prévu uniquement dans la t_parameters pour le moment mais non appliqué aux calculs).

Je viens de décrire les tables, vues et vues matérialsées. Je n'ai pas décrit les champs qui me semblent clairs par leur intitulé

@DonovanMaillard
Copy link
Contributor

DonovanMaillard commented Nov 10, 2020

Coté BDD je pense qu'on est presque bons. J'ai supprimé le champs 'exta-value" de la t_parameters qui ne sert à rien et j'ai fait une PR avec ces premiers éléments :)

Je me rends compte qu'il reste un point qui a été prévu mais pas implémenté : les règles sont configurables par id_organisme, mais cette notion d'organisme et de paramètres en fonction de l'organisme n'est pas du tout répercutée ailleurs. Pour le moment les paramètres s'appliquent sur l'instance de manière globale. Je le ajoute à la todolist.

Autre point à discuter à termes, l'idée de camille qui consistait à ne pas calculer les profils sur la base de la synthèse directmeent, mais sur la base d'une vue de la synthèse, pour que chacun puisse paramétrer les données qu'il veut prendre en compte ou non avec un maximum de souplesse. On pourra alors supprimer le paramètre qui précise le statut de validation des données à conserver dans les profils.

@DonovanMaillard
Copy link
Contributor

DonovanMaillard commented Nov 18, 2020

Bonjour,

Un point sur les avancées, coté base je pense qu'on est OK cette fois :

  • la notion d'organisme est supprimée dans les tables paramètres, qui n'était pas implémentée ailleurs
  • une vue v_synthese_for_profiles sert à alimenter les profiles et peut être filtrée de manière plus fine si besoin
  • un paramètre permet de définir les rangs (genre, espèce etc) pour lesquels les profils doivent être calculés
  • les scripts install_db et install_existing_db intègrent désormais les profils
  • un script migration est préparé, en ajoutant les profils ET en suppimant les VM et fonctions que Gil avaient faites
  • la doc est à jour

Je fais une PR pour tout ça

@camillemonchicourt
Copy link
Member

Le SQL de création du schéma de la BDD des profils de taxons est dispo sur : https://github.com/PnX-SI/GeoNature/blob/develop/data/core/profiles.sql

@camillemonchicourt
Copy link
Member

Profils de taxons intégrés dans la 2.9.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants