-
Notifications
You must be signed in to change notification settings - Fork 31
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
Support all possible values of service periods and exceptions #112
Comments
Working on it, I've noticed some logical problems:
"services": [
"Mo-Fr",
"2017-12-01-2018-02-28"
] Does it mean:
"services": [
"Mo-Su"
],
"exceptions": [
"Sa"
] could be changed to: "services": [
"Mo-Fr",
"Su"
] |
I therefore would suggest some changes to the Schedule specification:
What do you think? |
"services": [
"Mo-Fr",
"2017-12-01-2018-02-28"
] This would mean (in ISO terms) that it works Monday to Friday, but in the date range (desember, january, and february) would service daily. If the purpose of that tagging is to indicate additional service within a date-range, |
Yes, you are right, but as for now, |
@ialokim How is it going with this one? I notice the output currently does not contain exceptions, and I understand it is unresolved due to logic issues? |
First of all, glad to see our little workflow is used for more projects around the globe!
That's true, we didn't agree on a generic approach and I didn't put much time in this project lately. For what purpose do you want to use the |
As stated in the wiki,
services
andexceptions
should supportUp to now, only some of these values and only the
services
key are supported by hardcoding them in code.We should add a generic function checking for each:
Then, we should use that function to consume
services
andexceptions
and to generate the GTFS service_period with the correct exceptions.The text was updated successfully, but these errors were encountered: