Update mellowroute.json fixture for Dec 2025 data, and move Halsted to Emerald/Union in Bridgeport - #112
Open
jeancochrane wants to merge 2 commits into
Open
Update mellowroute.json fixture for Dec 2025 data, and move Halsted to Emerald/Union in Bridgeport#112jeancochrane wants to merge 2 commits into
mellowroute.json fixture for Dec 2025 data, and move Halsted to Emerald/Union in Bridgeport#112jeancochrane wants to merge 2 commits into
Conversation
jeancochrane
marked this pull request as ready for review
June 7, 2026 03:55
mellowroute.json fixture, and move Halsted to Emerald/Union in Bridgeportmellowroute.json fixture for Dec 2025 data, and move Halsted to Emerald/Union in Bridgeport
kalilsn
approved these changes
Jun 7, 2026
kalilsn
left a comment
Collaborator
There was a problem hiding this comment.
Agree with all of your thinking and IME these streets are chill.
One thought though: how about removing the mellow tag from 35th? I'm sure it's a commonly taken route because there aren't many other places to cross the river (which also means this shouldn't affect routing much) but I would really not describe any part of 35th as chill.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR makes two changes to the
mellowroute.jsonfixture:Updates the fixture to include OSM IDs for ways as of December 2025 (the most recent data import). I was working on updating these tags in the
drop-unimportant-sidewalksbranch, and the new tags eventually got deployed to staging/prod, but I never got around to merging in the new fixture since Use a buffered bounding box to restrict ways in routing query聽#90 ended up superceding thedrop-unimportant-sidewalksbranch and I forgot to update the fixture in that PR. I committed this version of the fixture in bf6c17a on this branch; the fixture in that commit should exactly match the current state of the prod and staging databases.Removes Halsted in Bridgeport from the
streettag, and tags Emerald and Union instead. I've been feeling wary of this stretch of Halsted for a while now, and Riley O'Neil's tragic killing yesterday has left me convinced that Halsted is not safe for biking between Pershing and the river. I replaced Halsted with Emerald/Union in 8cef92d.Here is the current version of the map on prod:
Here is that same area, re-tagged with the changes from this branch (Emerald/Union for north/south travel through Bridgeport, rather than Halsted):
I double-checked Emerald and Union from 26th St to Root St on Street View to confirm that all intersections are either signalized, or include an all-way stop sign. I have been taking these two streets regularly from 35th to 26th for the past few months and I feel comfortable recommending them as chill routes. I haven't personally ridden them from 35th to Root yet, but Street View seems to show basically the same street quality as the northern section.
Note that while the South Halsted protected lane technically goes as far north as Pershing, I stopped tagging it and switched over to Emerald/Union at Exchange (which quickly turns into Root, visible on the map screenshots above). This is because I felt the rail bridge underpass at ~4000 South to be dicey when we biked under it last week. Root/Exchange also seems marginally better than Pershing for the one-or-two-block hop that is required to transfer from Halsted to Emerald/Union.
I did not tag Halsted as a
route(orange on the map) even though it is technically a city-recommended route with painted lanes. I chose not to tag it because I dislike theroutetag, find it to be counterproductive, and plan to remove it soon.