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

FeedExpirationDateNotice doesn't make any difference between 7 days and 30 days expirations #930

Closed
gcamp opened this issue Jul 21, 2021 · 3 comments · Fixed by #1245
Closed
Assignees
Labels
bug Something isn't working (crash, a rule has a problem) GTFS Best Practices Used for Adding or changing rules that belong in the GTFS Best Practices status: Work in progress A PR that would close this issue has been opened.

Comments

@gcamp
Copy link

gcamp commented Jul 21, 2021

FeedExpirationDateNotice checks for both 7 days and 30 days expiration without any distinction.

I don't feel strongly on either solution but it should :

  • only check for 30 days expiration and not mention 7 days.
  • Have a warning for 7 days and info for 30 days.
@gcamp gcamp added the bug Something isn't working (crash, a rule has a problem) label Jul 21, 2021
@isabelle-dr isabelle-dr added the GTFS Best Practices Used for Adding or changing rules that belong in the GTFS Best Practices label Aug 3, 2021
@isabelle-dr
Copy link
Contributor

isabelle-dr commented Aug 3, 2021

Thanks for opening this issue @gcamp.
This rule is based on the GTFS Best Practices in Dataset Publishing & General Practices

At any time, the published GTFS dataset should be valid for at least the next 7 days, and ideally for as long as the operator is confident that the schedule will continue to be operated.
If possible, the GTFS dataset should cover at least the next 30 days of service.

What do you think of having two different notices for those thresholds?
It could open the door to defining different severity levels for each one with the profiles feature implementation (that will allow users to upgrade certain rules).

Also, there is a discussion in #886 on improving the way we measure the end date of a dataset. Currently, it's based on feed_info.feed_end_date, which is an optional field of an optional file.

@gcamp
Copy link
Author

gcamp commented Aug 6, 2021

I would agree that severity levels would be a good next step. There could have two notices but I don't think that would be very useful in practice.

@isabelle-dr isabelle-dr modified the milestones: v3.2.1 - rule improvements, v3.3.0 - rule improvements, Rule improvements Jan 9, 2022
@isabelle-dr isabelle-dr added good first issue Good task for newcomers to work on. and removed good first issue Good task for newcomers to work on. labels Jan 9, 2022
@isabelle-dr
Copy link
Contributor

Related issue: #1117

@KClough KClough self-assigned this May 26, 2022
@holly-g holly-g moved this to Prioritized Backlog (sorted) in Data Services: External Contributions Jul 1, 2022
@holly-g holly-g moved this from Prioritized Backlog (sorted) to Next Sprint in Data Services: External Contributions Jul 13, 2022
@holly-g holly-g moved this from Next Sprint to Current Sprint in Data Services: External Contributions Aug 16, 2022
@holly-g holly-g moved this from Current Sprint to Under Review in Data Services: External Contributions Aug 24, 2022
@isabelle-dr isabelle-dr added the status: Work in progress A PR that would close this issue has been opened. label Sep 15, 2022
Repository owner moved this from Under Review to Next Sprint in Data Services: External Contributions Sep 30, 2022
@holly-g holly-g moved this from Next Sprint to Under Review in Data Services: External Contributions Sep 30, 2022
@holly-g holly-g moved this from Under Review to Waiting for Release in Data Services: External Contributions Sep 30, 2022
@holly-g holly-g moved this from Waiting for Release to Completed in Data Services: External Contributions Feb 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working (crash, a rule has a problem) GTFS Best Practices Used for Adding or changing rules that belong in the GTFS Best Practices status: Work in progress A PR that would close this issue has been opened.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants