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

Flex: missing_pickup_booking_rule_id #1821

Closed
emmambd opened this issue Sep 9, 2024 · 3 comments · Fixed by #1966 or #2001
Closed

Flex: missing_pickup_booking_rule_id #1821

emmambd opened this issue Sep 9, 2024 · 3 comments · Fixed by #1966 or #2001
Assignees
Labels
flex Rules and rule changes related to GTFS-Flex. new rule New rule to be added

Comments

@emmambd
Copy link
Contributor

emmambd commented Sep 9, 2024

Describe the problem

See below

Describe the new validation rule

missing_pickup_booking_rule_id as defined in the table here

Sample GTFS datasets

No response

Severity

Warning

Additional context

No response

@emmambd emmambd added status: Needs triage Applied to all new issues new rule New rule to be added labels Sep 9, 2024
@emmambd emmambd added this to the 7.0 Validator Release milestone Sep 9, 2024
@emmambd emmambd added the flex Rules and rule changes related to GTFS-Flex. label Dec 2, 2024
@cka-y cka-y self-assigned this Feb 4, 2025
@cka-y cka-y linked a pull request Feb 4, 2025 that will close this issue
5 tasks
@emmambd
Copy link
Contributor Author

emmambd commented Mar 3, 2025

Modify this check to look for if start_pickup_drop_off_window/end_pickup_drop_off_window is defined rather than if booking_rules.txt exists

@emmambd emmambd reopened this Mar 3, 2025
@cka-y cka-y removed their assignment Mar 3, 2025
@emmambd emmambd added status: Ready An issue that is ready to be worked on. and removed status: Needs triage Applied to all new issues labels Mar 3, 2025
@qcdyx qcdyx self-assigned this Mar 3, 2025
@qcdyx qcdyx removed the status: Ready An issue that is ready to be worked on. label Mar 3, 2025
@qcdyx
Copy link
Contributor

qcdyx commented Mar 3, 2025

The notice description is pickup_booking_rule_id is recommended when pickup_type=2 and drop_off_booking_rule_id is
recommended when drop_off_type=2, shall I change it to start_pickup_drop_off_window should be defined when pickup_type=2 and end_pickup_drop_off_window should be defined when drop_off_type=2? @emmambd

@emmambd
Copy link
Contributor Author

emmambd commented Mar 3, 2025

Good catch @qcdyx ! Revised description:

pickup_booking_rule_id is recommended when pickup_type=2 and drop_off_booking_rule_id is
recommended when drop_off_type=2. Currently, this notice is only triggered on feeds when either start_pickup_drop_off_window or end_pickup_drop_off_window is defined, since this recommendation was added to the specification for feeds with GTFS-Flex.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flex Rules and rule changes related to GTFS-Flex. new rule New rule to be added
Projects
None yet
3 participants