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.

Sign up for GitHub

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

Jump to bottom

Problématique RTGE #182

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Problématique RTGE #182

wants to merge 5 commits into from

Conversation

romain-jault
Copy link
Contributor

Objectif

Mettre à disposition avec un flux WFS les données contenues dans les tables TA_LIG_TOPO_F et TA_POINT_TOPO_F

Actions à mener

Créer des tables pour accueillir les données sur l'instance de production,

  • TA_POINT_TOPO_F
  • TA_LIG_TOPO_F_ARCS_LINEARISES

Créer les vues de restitution des données avec l'altitude dans un champ si nécessaire:

  • VM_RTGE_POINT
  • VM_RTGE_LINEAIRE
  • VM_RTGE_SOMMET_LINEAIRE
  • un reviewer a été ajouté
  • le code et les noms d'objets suivent la syntaxe

@romain-jault
Problématique RTGE
f9307c9 
Requête pour créer les tables:
TA_POINT_TOPO_F et TA_LIG_TOPO_F_ARCS_LINEARISES
ainsi que les vues materialisees:
VM_RTGE_POINT
VM_RTGE_LINEAIRE
VM_RTGE_SOMMET_LINEAIRE
@romain-jault romain-jault added the Non intégrable En cours de travail, ne pas intégrer label Jul 13, 2022
@romain-jault
Update creation_vm_vm_rtge_sommet_lineaire.sql
f295457 
Changement du nom de la VM pour permettre l'auto-complétion dans les logiciels.
@romain-jault
Mise à jour de la branche avant le merge
1d18285 
Mise à jour des fichiers.
@romain-jault romain-jault added A merger Pull request pouvant être mergée selon son auteur and removed Non intégrable En cours de travail, ne pas intégrer labels Sep 2, 2022
@romain-jault
Creation des vues du projet RTGE
786abe8 
création de 4 vues:
- V_RTGE_ALTI
- V_RTGE_POINT
- V_RTGE_LINEAIRE
- V_RTGE_LINEAIRE_SOMMET
qui interrogent les tables:
- TA_RTGE_ALTI
- TA_RTGE_POINT
- TA_RTGE_LINEAIRE
- TA_RTGE_LINEAIRE_SOMMET
BaptisteJacq
BaptisteJacq reviewed Mar 24, 2023
View reviewed changes
Copy link
Contributor

@BaptisteJacq BaptisteJacq left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Problèmes identifiés :

Absence de Z dans les MTD spatiales des vues alors que les tables sur lesquelles elles tapent en ont un.

Remarques :

  • enlève tout ce qui est cudl des commentaires (inutile) ;
  • attention aux accents dans les commentaires ;
  • différences entre les commentaires du fichier et ceux de l'objet alors qu'ils sont censés être identiques ;

schema/donnees_geo/tables/creation_ta_rtge_alti.sql
DROP INDEX TA_RTGE_ALTI_CODE_TYPE_IDX;
DROP INDEX TA_RTGE_ALTI_LIBELLE_TYPE_IDX;
DROP TABLE TA_RTGE_ALTI;
*/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rajouter un commit ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est ajouté

schema/donnees_geo/tables/creation_ta_rtge_alti.sql Outdated
DROP INDEX TA_RTGE_ALTI_IDENTIFIANT_TYPE_IDX;
DROP INDEX TA_RTGE_ALTI_CODE_TYPE_IDX;
DROP INDEX TA_RTGE_ALTI_LIBELLE_TYPE_IDX;
DROP TABLE TA_RTGE_ALTI;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rajoute CASCADE CONSTRAINTS

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est ajouté

schema/donnees_geo/tables/creation_ta_rtge_lineaire.sql Outdated
DROP INDEX TA_RTGE_LINEAIRE_IDENTIFIANT_TYPE_IDX;
DROP INDEX TA_RTGE_LINEAIRE_CODE_TYPE_IDX;
DROP INDEX TA_RTGE_LINEAIRE_LIBELLE_TYPE_IDX;
DROP TABLE TA_RTGE_LINEAIRE;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rajoute CASCADE CONSTRAINTS et un commit

Copy link
Contributor Author

@romain-jault romain-jault Mar 24, 2023 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est ajoutés

schema/donnees_geo/tables/creation_ta_rtge_lineaire.sql Outdated


-- 2. Commentaire
COMMENT ON TABLE G_GEO.TA_RTGE_LINEAIRE IS 'Table qui présente les éléments contenus dans la table GEO@cudl.TA_LIG_TOPO_F. Les arcs sont linéarisés';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

enlève le @CUDL

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est retiré.

schema/donnees_geo/tables/creation_ta_rtge_lineaire_sommet.sql Outdated
DROP INDEX TA_RTGE_LINEAIRE_SOMMET_IDENTIFIANT_TYPE_IDX;
DROP INDEX TA_RTGE_LINEAIRE_SOMMET_CODE_TYPE_IDX;
DROP INDEX TA_RTGE_LINEAIRE_SOMMET_LIBELLE_TYPE_IDX;
DROP TABLE TA_RTGE_LINEAIRE_SOMMET;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rajoute CASCADE CONSTRAINTS et un commit.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est ajoutés

schema/donnees_geo/vues/creation_v_rtge_lineaire.sql Outdated
COMMENT ON TABLE G_GEO.V_RTGE_LINEAIRE IS 'Vue qui présente les points contenus dans la table G_GEO.TA_RTGE_LINEAIRE (les arcs sont linearises).';

-- 3. Creation des commentaires des colonnes.
COMMENT ON COLUMN G_GEO.V_RTGE_LINEAIRE.IDENTIFIANT_OBJET IS 'Identifiant interne de l''objet geographique - Cle primaire de la vue materialisee';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

il ne s'agit pas d'une VM mais d'une vue.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est corrigé

schema/donnees_geo/vues/creation_v_rtge_lineaire.sql Outdated
VALUES(
'V_RTGE_LINEAIRE',
'GEOM',
SDO_DIM_ARRAY(SDO_DIM_ELEMENT('X', 684540, 719822.2, 0.005),SDO_DIM_ELEMENT('Y', 7044212, 7078072, 0.005)),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pas de Z alors que les données source en ont un. C'est normal ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est corrigé

schema/donnees_geo/vues/creation_v_rtge_lineaire_sommet.sql
@@ -0,0 +1,73 @@
-- Creation de la vue V_RTGE_LINEAIRE afin de restituer les informations des dossiers
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commentaire différent de celui de la vue alors qu'il sont censés être identiques.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est corrigé

schema/donnees_geo/vues/creation_v_rtge_lineaire_sommet.sql Outdated
VALUES(
'V_RTGE_LINEAIRE_SOMMET',
'GEOM',
SDO_DIM_ARRAY(SDO_DIM_ELEMENT('X', 684540, 719822.2, 0.005),SDO_DIM_ELEMENT('Y', 7044212, 7078072, 0.005)),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pas de Z alors que les données source en ont un. C'est normal ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est corrigé

schema/donnees_geo/vues/creation_v_rtge_point.sql
VALUES(
'V_RTGE_POINT',
'GEOM',
SDO_DIM_ARRAY(SDO_DIM_ELEMENT('X', 684540, 719822.2, 0.005),SDO_DIM_ELEMENT('Y', 7044212, 7078072, 0.005)),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pas de Z alors que les données source en ont un. C'est normal ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est corrigé

@romain-jault
Correction coquille
96519d2 
Correction suite à une relecture de la pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BaptisteJacq BaptisteJacq BaptisteJacq left review comments

@Jean-Roc Jean-Roc Awaiting requested review from Jean-Roc

Assignees

@BaptisteJacq BaptisteJacq

Labels
A merger Pull request pouvant être mergée selon son auteur
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@romain-jault @BaptisteJacq