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

Samenbrengen van conceptuele schema's in één gemeenschappelijke verzamelplek #247

Closed
ArjanLoeffen opened this issue Apr 4, 2022 · 9 comments

Comments

@ArjanLoeffen
Copy link
Contributor

TLDR: Als we een XML schema willen genereren voor een instelling voegen we daar schema's bij die van elders zijn opgehaald, en lokaal worden bewaard. Dat doet iedereen, waardoor veel lokale kopieën ontstaan. Het is mogelijk die allemaal in één verzamelplek bij elkaar te plaatsen. Beheer wordt daardoor veel simpeler.

Uitwerking:

We hebben nu een constructie waarin per instelling wordt bepaald wat de XML schema's zijn (GML, Xlink, ISO Citations etc.). Deze schema's wordt per instelling beheerd, in:

/Imvertor-Chains/src/main/resources/input/[naam van instelling]/xsd

Doordat veel instellingen gelijke schema's gebruiken worden deze meermaals gedupliceerd. Dat is niet handig en levert potentieel afwijkingen op die niet behoren op te treden. Ook wordt de samenwerking in feite ondermijnd. Iedereen kan zijn/haar eigen ding doen. Bijvoorbeeld: welke constructies horen nu eigenlijk bij een GML 322 profiel?

Het is naar mijn mening mogelijk alle schema's samen met de conceptuele schema's te verzamelen in één verzamelplek (gewoon binnen dit GIThub project), waarnaar dan vanuit alle instellingen kan worden verwezen. Er treedt dan geen duplicatie meer op.

Een voorbeeld van de huidige situatie:

  • Kadaster maakt gebruik van GML322.
  • Het conceptuele schema en de mapping zit in '/input/Kadaster/xsd/conceptual-schemas.xml'.
  • Het bijbehorende XML schema (voor de Kadaster en ISO19136 schema samensteller) staat in /input/Kadaster/xsd/www.opengis.net/GML322-20160101/gml/3.2.2

Nieuwe situatie:

  • Kadaster maakt functioneel op dezelfde manier gebruik van GML322.
  • Een referentie naar het conceptuele schema en de mapping zit in '/input/Kadaster/xsd/conceptual-schemas.xml'.
  • Het bijbehorende XML schema (voor de Kadaster en ISO19136 schema samensteller) staan in /cs/xsd/www.opengis.net/GML322-20160101/gml/3.2.2
  • De nieuwe folder /cs (= verzamelplek) bevat alle conceptuele schema's, mappings, en XML schema's die nodig zijn voor het samenstellen van complete producten. (Dat kan zelfs uitgebreid worden in de toekomst.)

Effect op bestaande werkwijze:

  1. Aanpassingen aan de XML schema's doen we in de verzamelplek
  2. Aanpassingen aan de conceptuele schema's doen we in de verzamelplek
  3. De conceptual schema's van de instellingen bevatten alleen nog referenties naar conceptual schemas en maps in de repository (via xinclude).

Graag reactie, ik zal er ook achteraan gaan omdat we voor een IHW schema opdracht zitten.

@ArjanLoeffen ArjanLoeffen changed the title Samenbrengen van conceptuele schema's in één gemeenschappelijke repository Samenbrengen van conceptuele schema's in één gemeenschappelijke verzamelplek Apr 4, 2022
@ArjanLoeffen
Copy link
Contributor Author

Nb. komt in essentie neer op
1/ aanpassing van /Imvertor-Chains/src/main/resources/cfg/XsdCompiler/parms.xml:

<EXTERNAL_XSD_FOLDER>${system/inp-folder-path}/xsd</EXTERNAL_XSD_FOLDER>
wordt
<EXTERNAL_XSD_FOLDER>${system/managedinstallfolder}/cs</EXTERNAL_XSD_FOLDER>

2/ het verplaatsen van alle lokale folders
3/ het opdelen van alle conceptual schemas naar de nieuwe locaties, en het plaatsen van de nodige xinclude verwijzingen.

@kad-mesdat
Copy link

Nog wat vragen:

  1. Onze modelleurs merken hier niets van bij het genereren toch? Dit gaat over de organisatie binnen het imvertor project.
  2. Daar waar hierboven instelling genoemd wordt is dat in de betekenis van "setting" en niet Kadaster of VNG oid.
  3. Hoe doen we het dan met settings die afwijken? (Vermoeden: die wijzen naar het afwijkende conceptual schema in de ../cs/.. directory)
  4. Geldt het alleen voor conceptual schemas zoals GML, measure enz.

@ArjanLoeffen
Copy link
Contributor Author

ArjanLoeffen commented Apr 4, 2022

  1. Modelleurs merken niks. Het resultaat van deze verandering is onzichtbaar, het betreft alleen een aanpassing in de configuratie.
  2. Instelling is: Kadaster, IHW, Geonovum, BRO, Waarderingskamer. Deze zullen de aanpassen financieren (besluit dd. 26 sept 2022).
  3. Ja: Wanneer je afwijkt van de gemeenschappelijke configuratie, dan maak je je eigen conceptueel schema en mogelijk je eigen XML schema "folder"
  4. Het geldt voor alle conceptuele schema's zoals we die nu kennen.

ArjanLoeffen added a commit that referenced this issue Apr 9, 2022
Dit is een eerste stap naar het opbouwen van een verzamelplek voor alle
conceptuele schema's die worden gebruikt. (zie #247)

Minor, wordt alleen nog gebruikt door IHW.
ArjanLoeffen added a commit that referenced this issue Jul 6, 2022
Er zijn nu twee schema's voor relatierol leidend en relatiesoort
leidend.

Zie #247

Major.
@ArjanLoeffen
Copy link
Contributor Author

In de laatste UG is vastgesteld dat dit kan worden doorgevoerd; iedereen heeft akkoord gegeven voor financiering.

@ArjanLoeffen
Copy link
Contributor Author

ArjanLoeffen commented Nov 2, 2022

Is er nog iemand die GML 3.2.1 gebruikt (en dus niet GML 3.2.2 of een profiel daarvan) in een Imvertor run?

@HanWelmer
Copy link
Contributor

BRO gebruikt momenteel GML 3.2.1. Maar op basis van het document "07-036r1_Geography_Markup_Language_3.2.2_corrigendum" zie ik geen problemen om te upgraden naar GML 3.2.2.

@ThiesMesdag
Copy link
Contributor

Het Kadaster is volgens mij helemaal over op 3.2.2. Of zou dat moeten zijn.

@wilkoquak
Copy link
Collaborator

Is er nog iemand die GML 3.2.1 gebruikt (en dus niet GML 3.2.2 of een profiel daarvan) in een Imvertor run?

Omdat het 3.2.2 xsd in de 3.2.1 map staat op de GML schema downloadpagina en het xsd in de namespace 3.2 rondhangt is de vraag niet hemelaal helder. Door deze verwarring weet volgens mij niemand precies wat je bedoelt en is het wat mij betreft OK om de meest recente versie (=3.2.2) te pakken. Overigens zou het wellicht logischer zijn om te zeggen dat we GML 3.2 gebruiken en dan altijd de meest recente te pakken.

ArjanLoeffen added a commit that referenced this issue Nov 4, 2022
Deze komen in één gemeenschappelijke verzamelplek. Dat betekent dat alle
schema's een echt goede plek moeten krijgen; er onstaan nieuwe owners
zoals OGC, OMG, ISO etc. Binnen die owners worden de conceptuele
schema's opgenomen, waarnaar vanuit configuraties wordt verwezen
(xinclude).

Eerste stap in #247

Minor. De werkzaamheden hebben geen zichtbaar effect op de werking van
Imvertor.
ArjanLoeffen added a commit that referenced this issue Nov 4, 2022
In deze slag zijn voor het Kadaster als werk-casus de configuraties
opgesteld. Work in progress op #247

Bugfix, refactoring.
ArjanLoeffen added a commit that referenced this issue Nov 8, 2022
Nieuwe stap in #247 - alle conceptuele schemas zijn onder een owner
opgenomen, en de configuratie is klaar om getest te worden op basis van
aangeleverde regressietestgevallen.

De configuraties zijn ook opgetild naar NEN3610-2022 en MIM111. Hierdoor
kunnen major verschillen ontstaan.
@melsk-r
Copy link
Contributor

melsk-r commented Feb 7, 2023

Zie #309.

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

8 participants