-
Notifications
You must be signed in to change notification settings - Fork 11
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
GLX stops, Medford Branch #79
Conversation
Hi Devin, thanks for the PR! We have been talking about doing this forever so I appreciate you getting a jump on this as GLX approaches. I think we'll want to deploy this in two stages:
Would you be amenable to implementing this? We could launch early with both new segments if we find a way to make it clear that they're not open yet (opacity or dotted lines?) or we can trickle them out as they come online. Thanks again for your work on this! |
That sounds good. I'll take another crack at this one night this week. |
Hey Devin, any updates on this? Union branch is getting close, and we're looking forward to seeing your changes 😃 |
@mathcolo Sorry I've been out of the country so I haven't had a chance to do this. I'll plan to update it next week as I arrive back tomorrow. However if someone else wishes to take over sooner I understand. I'm adding time to my calendar now to look at it |
@devinmatte Sounds good! Just wanted to make sure you still had it on your mind since we're close to the opening. Keep us posted if you need us to take over 👍 |
The place ids are placeholders are the mbta API doesn't seem to yet have the new stations listed as available. Due to the lines being at the oposite end, I had to do some adjustments with curves before the shared stops
So I'm currently unable to understand why Lechmere doesn't show a label despite the API returning a stop name for it, and it being in the labeled list. Maybe there's some code I've missed, but the attached image from above is what I'm able to get with the current state of the PR |
When I checked just now, Lechmere and Science Park are still missing from the API:
So I think we gotta give it a few more days before they come back online and get labels. This is looking great, though! Between now and Monday, if you get the dashed lines working, I think we can launch this. Otherwise let's cut the Medford branch out for now and add Science Park, Lechmere, and Union to the trunk. |
I'll split this into two PRs since it looks like dashed lines are more complicated than I have time to dedicate to this |
Will you be able to get a PR for the Union branch out today? No worries either way. |
Yep, I was planning to split this this afternoon. Sorry for the delay |
I see there was already a commit made, sorry I didn't get to this. I'll make sure this PR is fully ready to merge day 0 for the remaining stops |
Waiting for real API place names, but marking ready for review in the meantime |
Still thinking Union should be a stub and Medford should continue straight |
Reposting this from Slack with the new station IDs: https://github.com/mbta/api/pull/518/files
I'm in favor of this with a potential caveat: our map of the GL network is already fairly wide, and the four southern branches fan out to take up essentially the entire viewport of small phones. So we should avoid making the bounding box of the entire system wider or it will get cut off. Since the trunk currently continues into C branch, we can mirror the curve of the B branch on the north side to create the Union stub (mocked up here): But, we will want to fit the station label in a box to the right of the station, in the cramped space indicated by the white rectangle above. So we may have to abbreviate it to |
The place ids are placeholders are the mbta API doesn't seem to yet have the new stations listed as available.
Due to the lines being at the opposite end, I had to do some adjustments with curves before the shared stops to deal with the new D branch. No idea if this is how mantainers would want this to be, but thought this was a good starting point.
Relates to #58