Skip to content

New better timetable example from Slovenia#765

Open
atibaut wants to merge 4 commits into
TransmodelEcosystem:v2.1-wipfrom
atibaut:timetable_example_Slovenia
Open

New better timetable example from Slovenia#765
atibaut wants to merge 4 commits into
TransmodelEcosystem:v2.1-wipfrom
atibaut:timetable_example_Slovenia

Conversation

@atibaut

@atibaut atibaut commented Aug 29, 2024

Copy link
Copy Markdown
Contributor

This example of a timetable replaces the file Netex_09.2_Bus_SimpleTimetable_Slovenia.xml. The timetable was created from a national NeTEx service in Slovenia (Ministry of Transport/NCUP).

@Aurige

Aurige commented Aug 29, 2024

Copy link
Copy Markdown
Contributor

fine for me, but maybe put if in NeTEx\examples\standards\slovenia rather than in NeTEx\examples\functions\timetable ?

@skinkie

skinkie commented Sep 4, 2024

Copy link
Copy Markdown
Contributor

@atibaut my suggestion would be to change pos to posList. That would already save a considerable amount of data.
I notice the namespace "p10:" please define this at the top level of the document als gml:

                           <LineString
                                p10:id="SI_SI0_LineStringType_00000000-1177-000a-0011-236e001135bb_IJPP"
                                xmlns:p10="http://www.opengis.net/gml/3.2"
                                xmlns="http://www.opengis.net/gml/3.2">

I notice that ScheduledStopPointRef, ServiceLinkRef, OperatingDayRef, DayTypeRef all lack the version attribute, that this is not being picked up, is a bug in the XML Schema.

                                <StopPointInJourneyPattern
                                    id="SI:SI0:StopPointInJourneyPattern:00000000-1177-0009-0000-000000725b39:IJPP"
                                    created="2023-10-06T04:51:58.9174842Z" version="1" order="7">
                                    <ScheduledStopPointRef
                                        ref="SI:SI0:ScheduledStopPoint:00000000-1177-0003-0000-00000011258e:IJPP"
                                    />
                                <ServiceLinkInJourneyPattern
                                    id="SI:SI0:ServiceLinkInJourneyPattern:00000000-1177-0009-0072-5b4100725b40:IJPP"
                                    created="2023-10-06T04:51:58.9174842Z" version="1" order="14">
                                    <ServiceLinkRef
                                        ref="SI:SI0:ServiceLink:00000000-1177-0003-0011-372900113744:IJPP"
                                    /> 
                                </ServiceLinkInJourneyPattern>
                        <DayTypeAssignment
                            id="SI:SI0:DayTypeAssignment:00000000-1177-0005-0000-062400010e54:IJPP"
                            created="2023-10-26T08:11:03.7694927Z" version="2" order="345">
                            <OperatingDayRef
                                ref="SI:SI0:OperatingDay:00000000-1177-0005-0000-01b70000594d:IJPP"/>
                            <DayTypeRef version="1"
                                ref="SI:SI0:DayType:00000000-1177-0004-0000-06240000065e:IJPP"/>
                        </DayTypeAssignment>

@nick-knowles

Copy link
Copy Markdown
Contributor

This is not true the attribute is present but optional. because you may choose to have versionRef or version defepending on whether it is an internal or external ref.

@ue71603 ue71603 added the change-request Request for an additional feature -- Must be discussed with the group label Sep 10, 2024
@ue71603
ue71603 self-requested a review September 10, 2024 12:35
@ue71603

ue71603 commented Sep 24, 2024

Copy link
Copy Markdown
Contributor

@atibaut pls update the PR

@Aurige Aurige added this to the netex_2.0 milestone Sep 27, 2024
@skinkie skinkie removed this from the netex_2.0 milestone Apr 3, 2025
Copilot AI review requested due to automatic review settings July 10, 2026 22:00
@atibaut
atibaut force-pushed the timetable_example_Slovenia branch from 91938c9 to c5e1d48 Compare July 10, 2026 22:00

This comment was marked as resolved.

@atibaut
atibaut changed the base branch from v1.3 to v2.0 July 10, 2026 22:00
@atibaut

atibaut commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

It is never too late to improve a NeTEx timetable example. 😀

PR updated and rebased onto v2.0 (base retargeted from v1.3):

  • Moved to examples/standards/slovenia/ as suggested by @Aurige.
  • As suggested by @skinkie, the repeated per-LineString p10:/default namespace declarations are gone, and gml: is now declared once on the root element, and the ~2 700 individual gml:pos points are merged into one gml:posList per LineString (with srsName, srsDimension and count); the file shrank from 12376 to 9591 lines.
  • File extension lowercased to .xml, and the CI validation script's find -name "*.xml" is case-sensitive and would have skipped the uppercase .XML.
  • xsi:schemaLocation now uses the conventional relative path instead of an absolute local path.

Validated against NeTEx_publication.xsd on current v2.0 with xmllint.

Note: the original Netex_09.2_Bus_SimpleTimetable_Slovenia.xml teaching example is left in place, since this file now lives under examples/standards/ as a real-world NAP export rather than replacing the simple functional example.

@atibaut
atibaut force-pushed the timetable_example_Slovenia branch from b08e9a4 to 884cf8b Compare July 10, 2026 22:12
Updated per review: moved to examples/standards/slovenia, lowercase .xml
extension so CI validation finds it, gml namespace declared once on the
root element, pos points merged into gml:posList per LineString, and
xsi:schemaLocation made relative.
@atibaut
atibaut force-pushed the timetable_example_Slovenia branch from 884cf8b to 8b38dd2 Compare July 10, 2026 22:14
@atibaut atibaut added the example An additional example label Jul 10, 2026
@atibaut

atibaut commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

@skinkie @TuThoThai the updated example is ready for your review.

@atibaut

atibaut commented Jul 20, 2026

Copy link
Copy Markdown
Contributor Author

Hi @TuThoThai can someone look at this. 😀

ue71603
ue71603 previously approved these changes Jul 20, 2026
@TuThoThai

Copy link
Copy Markdown
Contributor

@atibaut, I am moving the base branch to v2.1-wip since it is not a bug-fix. 😄

@TuThoThai
TuThoThai changed the base branch from v2.0 to v2.1-wip July 20, 2026 12:43
@TuThoThai
TuThoThai dismissed ue71603’s stale review July 20, 2026 12:43

The base branch was changed.

@TuThoThai TuThoThai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@atibaut, I made some comments in the file. Please note that for some comments, e.g. the position of version, my comments are only in the first occurrence and not in everyone of them.

<Name>Petrina - Kočevje (1003)</Name>
<TransportMode>bus</TransportMode>
<PublicCode>1003</PublicCode>
<PrivateCode>51314</PrivateCode>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@atibaut, please note that PrivateCode has been depcreated since v2.0 in favour of privateCodes in DataManagedGroup

<Route id="SI:SI0:Route:00000000-1177-0008-0000-00000004b3b4:IJPP" created="2023-10-06T04:51:58.9174842Z" version="1">
<Name>Kočevje 1 7003</Name>
<LineRef ref="SI:SI0:Line:00000000-1177-0007-0000-00000000c872:IJPP"/>
<pointsInSequence>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@atibaut, in all the pointsInSequence, could you please remove order as it was deprecated in favour of XML ordering?

<OperatorRef ref="SI:SI0:Operator:00000000-1177-0001-0000-00000000045f:IJPP"/>
<passingTimes>
<TimetabledPassingTime id="SI:SI0:TimetabledPassingTime:00000000-1177-0009-0000-000000725b33:IJPP" created="2023-10-06T04:51:58.9174842Z" version="1">
<StopPointInJourneyPatternRef version="1" ref="SI:SI0:StopPointInJourneyPattern:00000000-1177-0009-0000-000000725b33:IJPP"/>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@atibaut, for all the StopPointInJourneyPatternRef, could we have version at the end to be consistent in the file?

</Line>
</lines>
<scheduledStopPoints>
<ScheduledStopPoint id="SI:SI0:ScheduledStopPoint:00000000-1177-0003-0000-00000011236e:IJPP" created="2023-10-06T04:51:58.9174842Z" version="1">

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@atibaut, I believe that in this file all your ScheduledStopPoint are associated with a Quay via PassengerStopAssignment. So, name could be removed as redundant

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change-request Request for an additional feature -- Must be discussed with the group example An additional example

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants