Skip to content
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

Merged
merged 12 commits into from
Dec 11, 2022
Merged

GLX stops, Medford Branch #79

merged 12 commits into from
Dec 11, 2022

Conversation

devinmatte
Copy link
Member

@devinmatte devinmatte commented Feb 14, 2022

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

Screen Shot 2022-07-31 at 5 14 35 PM

@devinmatte devinmatte changed the title GLX stops, initial [#58] GLX stops, initial Feb 14, 2022
@devinmatte devinmatte changed the title [#58] GLX stops, initial GLX stops, initial Feb 14, 2022
@idreyn
Copy link
Contributor

idreyn commented Feb 15, 2022

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:

  • One to add the Science Park, Lechmere, and Union segment opening in March. We were thinking of putting the Union stop on the same straight line as the GL trunk for simplicity's sake. It could also dip out a bit the same way the rest of the branches do, but the line should be running vertically at Union so the station can be labelled properly.
  • Another to add the Medford branch, scheduled for May (but probably opening later). Our plan is to have this branch dip to the right of the trunk with a wiggle(). Together the two branches would look like this (apologies for the worst mockup you've ever seen):

image

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!

@devinmatte
Copy link
Member Author

That sounds good. I'll take another crack at this one night this week.
I'll look into changing opacity/dotting lines for the unopened branch, if I can't get something clean I'll open two separate PRs

@mathcolo
Copy link
Contributor

mathcolo commented Mar 4, 2022

Hey Devin, any updates on this? Union branch is getting close, and we're looking forward to seeing your changes 😃

@devinmatte
Copy link
Member Author

@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

@mathcolo
Copy link
Contributor

mathcolo commented Mar 5, 2022

@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 👍

@devinmatte
Copy link
Member Author

Screen Shot 2022-03-13 at 9 10 26 PM

👋 I just got this working as designed above. Should have the PR updated tonight. Going to play around with dashed lines and see if the mbta api has proper labels yet

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
@devinmatte
Copy link
Member Author

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

@idreyn
Copy link
Contributor

idreyn commented Mar 16, 2022

When I checked just now, Lechmere and Science Park are still missing from the API:

https://api-v3.mbta.com/stops?filter[route]=Green-E

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.

@devinmatte
Copy link
Member Author

I'll split this into two PRs since it looks like dashed lines are more complicated than I have time to dedicate to this

@idreyn
Copy link
Contributor

idreyn commented Mar 20, 2022

Will you be able to get a PR for the Union branch out today? No worries either way.

@devinmatte
Copy link
Member Author

Yep, I was planning to split this this afternoon. Sorry for the delay

@devinmatte
Copy link
Member Author

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

@devinmatte devinmatte changed the title GLX stops, initial GLX stops, Medford Branch Jul 31, 2022
@nathan-weinberg
Copy link
Member

I feel like Medford branch should be the main one and Union should wiggle off? Similar to the map they have posted in some of the trolleys
image

@nathan-weinberg nathan-weinberg linked an issue Nov 17, 2022 that may be closed by this pull request
@devinmatte devinmatte marked this pull request as ready for review November 19, 2022 02:51
@devinmatte
Copy link
Member Author

Waiting for real API place names, but marking ready for review in the meantime

@nathan-weinberg
Copy link
Member

Still thinking Union should be a stub and Medford should continue straight

@idreyn
Copy link
Contributor

idreyn commented Dec 5, 2022

Waiting for real API place names, but marking ready for review in the meantime

Reposting this from Slack with the new station IDs: https://github.com/mbta/api/pull/518/files

Still thinking Union should be a stub and Medford should continue straight

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):

image

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 Union instead of Union Square — which I think is absolutely fine.

@mathcolo mathcolo merged commit 290a300 into transitmatters:master Dec 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extend Green Line visualization/tracking to account for GLX
4 participants