NeTEx-CEN / NeTEx

NeTEx is a CEN Technical Standard for exchanging Public Transport schedules and related data.
http://netex-cen.eu
GNU General Public License v3.0
80 stars 39 forks source link

TrainElementType vs TrainElement #703

Open ue71603 opened 6 months ago

ue71603 commented 6 months ago

for accessibility we need the concrete elements as well.

nick-knowles commented 5 months ago

Are there three separate questions here? (a) How should we handle renaming of TrainElement to TrainElementType ? (As an additional deprecated element?).
(b) Do we need to implement train composition elements as below so can distinguish between types and instances. This has been added to Transmodel and NEtex UML but tot to XML yet as wasnt sure if specific train was really netex or / SIRI content c) Should the carriages and wagons just go in Resource frame as additional "Vehicles" or should image

ue71603 commented 3 months ago

@nick-knowles I thought mostly about b). c) is not a complete sentence. So I am not sure, if it is affected. a) is not that important to me...

ue71603 commented 3 months ago

Other problem: the new TrainElementType would contain the TrainElementGroup which contains an element TrainElementType which is an enumeration. Sorry, not thought trough.

ue71603 commented 3 months ago

I think also, that was decided in Transmodel is in direct conflict with the new SIRI European profile. image

I would like to see from you a picture of a Train with two TrainCompounds with 3 carriages each and then label, how this would need to be done with Type and concrete TrainElements (with EVN numbers). Currently I think it is overengineered and the CompoundTrain TrainComponent etc is misleading at best.

And for demonstration the problem with TrainElementType within TrainElementType:

5th Carriage carriage standardClass

If anything I would have prefered:

for type based data: <TrainComponent ... <TrainElementType ....

for real vehicles <TrainComponent ... <TrainElement .....

ue71603 commented 3 months ago

Strangly enough TrainElementTypeGroup contains "equipments". Which can't be on a Type.

The renaming just doesn't work.

I guess we need really:

TrainElement (abstract)

TrainElementType inherits from TrainElement ConcreteTrainElement inherits from TrainElement

We have TrainElementTypeGroup (which contains TrainElementTypeGroup and no equipments) and we have ConcreteTrainElementGroup (which contains TrainElementTypeGroup and the equipments and the EVN number or at least a note, where to store it, if it exists)

Also: currently DECK PLAN only associates with the TrainElementType. I hope this is working for the use cases.

Aurige commented 3 months ago

It's a pretty long and complex discussion with a lot of inputs coming from rail people, and requires a synchronisation with TM, and SIRI ... My main question is "can we stay with what we have ?" ... I'm not sure that we really have a instance related use case for NeTEx (probably more in the SIRI side). If so, the fact that currently DECK PLAN only associates with the TrainElementTypeRef, need to be documented (or renamed to TrainElementRef) since our current TrainElementType is an enum (TrainElementTypeTypeEnumeration) which cannot be refered.

ue71603 commented 3 months ago

What should the current situation be? TrainComposition is not implemented yet (and I oppose it). Other stuff from DeckPlan is already integrated. And I think that TrainElement -> TrainElementType is in principle also be implemented already.

Aurige commented 3 months ago

The TRAIN COMPOSITION is "A specific instance of a TRAIN composed of TRAIN COMPOSITION COMPONENTs in a certain order.", that's the " instance related use case" I'm mentioning. DeckPlan is Ok and independent (currently possibly assigned to VehicleType and specialisations, TrainComponent, TrainElement(Type) Journey or journeyPart). Isn't it enough for now ?

ue71603 commented 3 months ago

My problem is, that this is exactly what TrainElement did (and does in Siri) before some people thought they needed a TrainElementType and then tried to rename TrainElement. This results in Siri/OJP being no longer aligned with Transmodel. I

Aurige commented 3 months ago

So you are against the renaming in Transmodel, right ?