Skip to content

Remove Open Traffic prototype code - #2698

Merged
drewda merged 4 commits into
masterfrom
remove-opentraffic
Jan 7, 2019
Merged

Remove Open Traffic prototype code#2698
drewda merged 4 commits into
masterfrom
remove-opentraffic

Conversation

@drewda

@drewda drewda commented Dec 13, 2018

Copy link
Copy Markdown
Contributor

To be completed by pull request submitter:

To be completed by @opentripplanner/plc:

  • reviews and approvals by 2 members, ideally from different organizations
  • before merging: add a bullet point to the changelog file with description and link to the linked issue
  • after merging: update the relevant card on the roadmap

@drewda
drewda requested a review from a team December 13, 2018 23:44
@drewda

drewda commented Dec 13, 2018

Copy link
Copy Markdown
Contributor Author

@opentripplanner/plc I believe I've removed all dangling references -- but would appreciate another set of eyes.

@abyrd abyrd left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @drewda, I just reviewed these changes and they look good to me. I would agree that we should remove this outdated and unmaintained code, and these commits should do the job. We may want to retain some kind of mechanism for pluggable street speed sources, independent of the opentraffic-specific implementation. But that should be easy to add back in if and when it's needed.

@abyrd

abyrd commented Jan 7, 2019

Copy link
Copy Markdown
Member

@t2gran and @gmellemstrand I tagged you on this PR because we'll need one more review, and you've generally been favorable to removing dead code and reducing complexity so you may be motivated to review this.

@t2gran t2gran left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am in favor. I looked over the code changes and they looked straight forward.

@drewda
drewda merged commit 36d7a6a into master Jan 7, 2019
@drewda
drewda deleted the remove-opentraffic branch January 7, 2019 22:05
@Patafix

Patafix commented Jan 31, 2019

Copy link
Copy Markdown

Hello,
What are the alternative to have traffic on OTP after the 1.4.0

My team use this feature a lot.

Thanks

@abyrd

abyrd commented Jan 31, 2019

Copy link
Copy Markdown
Member

Hi @Patafix, the version of OpenTraffic included in OTP was a prototype that was never maintained and development moved on to a completely different format. We were planning to remove the code entirely. If you need it, you would need to create a branch and revert these commits.

Is it the OpenTraffic data format you're using or just the idea of adding speed data to OTP? It should be straightforward to create your own system like StreetSpeedSource.

@Patafix

Patafix commented Jan 31, 2019

Copy link
Copy Markdown

Thanks for you reply @abyrd we can use any format we want.

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.

5 participants