Skip to content

EPIAP: PlatformHeight introduced on StopPlaceSpace#295

Closed
skinkie wants to merge 1 commit into
TransmodelEcosystem:masterfrom
skinkie:epiap-platformheight-stopplacespace
Closed

EPIAP: PlatformHeight introduced on StopPlaceSpace#295
skinkie wants to merge 1 commit into
TransmodelEcosystem:masterfrom
skinkie:epiap-platformheight-stopplacespace

Conversation

@skinkie
Copy link
Copy Markdown
Contributor

@skinkie skinkie commented Feb 16, 2022

No description provided.

@skinkie skinkie added the enhancement non semantic enhacement: technical enhancement, etc. label Feb 16, 2022
@skinkie skinkie requested review from Aurige and ue71603 February 16, 2022 10:30
@skinkie skinkie requested a review from Joostb61 February 16, 2022 10:43
@Aurige
Copy link
Copy Markdown
Contributor

Aurige commented Feb 16, 2022

Shouldn't it be on Quays ? StopPlaceSpace can be waiting rooms, corridors, etc. I since it is an enhancement it should go to "next"

@skinkie
Copy link
Copy Markdown
Contributor Author

skinkie commented Feb 16, 2022

@Aurige we already had this discussion. We want to have in Quays, BoardingPositions, etc. If there should be introduced a different substitiongroup to facilitate enhancements that only relate to things along side of the track, that should be modelled first upstream.

@Aurige
Copy link
Copy Markdown
Contributor

Aurige commented Feb 16, 2022

Yes I understood that this information needs to be shared between Quays and BoardingPositions, but that's not a reason to spread it to other unrelated object. I would recommend doing an XSD Group , as it is most usually done in NeTEx, and use that group in both objects. Another to go could be to detail the restriction on AccessSpace and VehicleStoppingPlace, but that looks a bit unnatural too me.

@ue71603 ue71603 requested a review from duexw February 17, 2022 13:28
@ue71603
Copy link
Copy Markdown
Contributor

ue71603 commented Feb 17, 2022

May I ask you @skinkie and @Aurige to resolve this? If modeling in a group occurs, who does it?

@ue71603
Copy link
Copy Markdown
Contributor

ue71603 commented Mar 16, 2022

Is replaced by: #298

@ue71603 ue71603 closed this Mar 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement non semantic enhacement: technical enhancement, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants