-
Notifications
You must be signed in to change notification settings - Fork 3
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
added neighborhood and transit info to routes #21
base: main
Are you sure you want to change the base?
Conversation
On the problem that "for some modes (buses), the fact of there being a stop is not that useful on its own": Maybe we can treat this as metrics of a transit isochrone. Try to capture how much is reachable for a particular time budget from a given start point (e.g. area of a shade in the isochrone) or perhaps reachability from some number of anchor locations across Seattle that we care about (e.g. whether the anchor point is within some threshold color on the isochrone, like is it possible to reach CSE or Queen Anne from this location in less than 1 hour via transit?). This requires route+schedule knowledge, but there are APIs that sell that. Isochrones API |
ba708b9
to
f5c72be
Compare
@nickswalker added a reachability score 1-3 based on max google maps public transit time between the location and a fixed set of locations (temporarily, CSE, green lake, beacon hill) |
TODO:
Also, considering merging the locs and routes folders, since a lot of logic will probably end up shared. Thoughts @nickswalker ? |
Yes, would merge those folders |
47fb51e
to
edd3667
Compare
6d793e1
to
a9fd703
Compare
@hadasah Nice!! FWIW, I'd like to keep all the location info in the locs table and use the Not sure about best way to handle "neighborhoods traversed". Do we need that for scheduler? Can we recompute on demand? After this merges, I plan to reorganize the routes db into a directory of geojson and compute the table from those files. |
Handle special case where Tacoma Dome has same stop ID in both dirs Fix an issue where popups were getting insta-cleared due to a bug where the wrong exchange coords were checked
Please enter the commit message for your changes. Lines starting
@nickswalker @ztatlock ready to merge |
haven't changed the db.csv file yet, kept the new file separate for comparison temporarily