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

Hoogtelijnspecificatie.hoogtelijntype verplicht maken (CVGG093 en SDIMG-59) #164

Closed
PB-GNM opened this issue Mar 1, 2022 · 6 comments
Closed
Assignees
Labels
9 enhancement New feature or request SDIMG-59 Komt voort uit SDIMG-59 (#150) verwerkt in v2.0 verwerkt in versie 2.0 van IMG

Comments

@PB-GNM
Copy link
Collaborator

PB-GNM commented Mar 1, 2022

Aanleiding

Wijzigingsverzoek dat voortkomt uit #150 om het aantal regels in het model te verminderen.
Het gaat om de regel dat Hoogtelijnspecificatie.hoogtelijntype voor moet komen als Hoogtelijnspecificatie.hoogtelijnobjecttype er is.

image

Door Hoogtelijnspecificatie.hoogtelijntype verplicht te maken kan de regel vervallen. Als ze er beide niet zijn is er ook geen sprake van een objecttype Hoogtelijnspecificatie, maar alleen van een objecttype Hoogtelijn. Een andere oplossing is beschreven in het uiteindelijke wijzigingsverzoek zoals hieronder aangegeven.

Wijzigingsverzoek

  • Verplaats het attribuut hoogtelijntype van HoogtelijnSpecificatie naar Hoogtelijn. Dit kenmerk blijft optioneel en kan nog steeds de volgende waardes hebben (ballastbedlijn, kantaardebaanlijn, teentaludlijn, referentiepuntenlijn, tunnelbaklijn)
  • Verwijder de hele klasse HoogtelijnSpecificatie inclusief de regel die daar in voorkomt.

Het resulterende model ziet er dan als volgt uit:
image

Impactanalyse

  • Deze wijziging heeft alleen impact op partijen die hoogtelijnen aanleveren: Prorail en RWS
  • Deze wijziging heeft geen impact op de semantiek van het model
  • De wijziging resulteert wel in niet backwards-compatible wijziging in het xml-schema.
@PB-GNM PB-GNM added 9 enhancement New feature or request SDIMG-59 Komt voort uit SDIMG-59 (#150) labels Mar 1, 2022
@wilkoquak
Copy link
Member

wilkoquak commented May 24, 2022

Vandaag heeft een voorbespreking met Prorail plaatsgevonden. Deze zeggen het attribuut hoogtelijnobjecttype niet te gaan gebruiken terwijl zij de enige beoogde gebruiker zijn van dit attribuut. Dit attribuut kan dus geschrapt worden. Dit maakt de modellering nog eenvoudiger. Ik heb het voorstel aangepast conform deze informatie.

@wilkoquak
Copy link
Member

Uit voorbespreking met RWS blijkt dat ook zijn hoogtelijnobjecttype niet gebruiken. Het attribuut hoogtelijntype kan als optioneel kenmerk aan hoogtelijn worden toegevoegd.

@PB-GNM PB-GNM self-assigned this Jun 2, 2022
@PB-GNM
Copy link
Collaborator Author

PB-GNM commented Jun 7, 2022

Advies van adviesgroep 30-5-2022: wijziging doorvoeren.

@PB-GNM
Copy link
Collaborator Author

PB-GNM commented Jun 7, 2022

De wijziging is in versie 2.0 verwerkt zoals beschreven in het aangepaste wijzigingsverzoek van #164 (comment) .

@PB-GNM PB-GNM added the verwerkt in v2.0 verwerkt in versie 2.0 van IMG label Jun 7, 2022
@wilkoquak
Copy link
Member

Ook de regel verwijderd die zegt dat bij wegen alleen: kantaardebaanlijn, teentaludlijn en referentiepuntenlijn toegestaan zijn.

@wilkoquak
Copy link
Member

Met publicatie van release def-im-img-20220705 kan dit issue worden afgesloten.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
9 enhancement New feature or request SDIMG-59 Komt voort uit SDIMG-59 (#150) verwerkt in v2.0 verwerkt in versie 2.0 van IMG
Projects
None yet
Development

No branches or pull requests

2 participants