-
Notifications
You must be signed in to change notification settings - Fork 0
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
Naming convention: Types of routes #21
Comments
Oh man, I wish GitHub supported issues at an organization level. It might be worth making some sort of distinct knowledge base for things like this, though I think it makes sense to talk about it here, since Shark is the provider and essentially determines naming/type conventions. I think a more immediate step would be to start writing wiki pages for things like this (similar to the WAMP Events and RPCs page). That said, I think renaming |
The change improves semantics, making it more obvious that the list of stations in the attribute is the full, *ordered* list of stations that a Route will pass through, as opposed to just the set of stations it touches (which is what `Shark::Route#associated_objects[Shark::Station]` represents).
Closed by 5a416ce. |
I noticed in #19 that our terminologies for the varying route:station representations are drifting. Here's what I've been using:
Route.stations
route.associated_objects[Shark::Station]
Route.itinerary
route.stations
CanonicalRoute.stations
The main thing I see here that's somewhat confusing is how "itinerary" to Proper is called "stations" to Shark. If it's alright with you, and you like the terms I'm using enough, I propose renaming
route.stations
toroute.itinerary
.Additionally, Proper's
Route.stations
could be renamed toRoute.associatedStations
to reduce ambiguity, but it's typed as aSet
, which makes it clear it doesn't imply any order like an itinerary does.Let me know what you think, and if there's a better place for system design discussions not directly related to any application.
The text was updated successfully, but these errors were encountered: