marcusyoung / opentripplanner

An API wrapper for OpenTripPlanner (OTP) as an R package
11 stars 5 forks source link

feat: DSC changes #15

Closed mshodge closed 6 years ago

mshodge commented 6 years ago

Changes from the Data Science Campus, including:

We have tested this on our own unit tests and each function passed:

file test nb failed skipped error warning user system real
1 test_GTFS.R Test GTFS 1 0 FALSE FALSE 0 0.181 0.021 0.756
2 test_otpChoropleth.R Test otpChoropleth 1 0 FALSE FALSE 0 0.080 0.005 0.213
3 test_otpConnect.R Test otpConnect 1 0 FALSE FALSE 0 0.001 0.000 0.002
4 test_otpIsochrone.R Test otpIsochrone 1 0 FALSE FALSE 0 2.202 0.181 8.984
5 test_otpTripDistance.R Test otpTripDistance 1 0 FALSE FALSE 0 0.024 0.000 0.085
6 test_otpTripTime.R Test otpTriptime 1 0 FALSE FALSE 0 0.017 0.000 0.095

These changes were made to develop a series of R scripts for certain tasks required at the Data Science Campus. If the changes are not passable (rejected) please allow us to use your original code with acknowledgements. If the changes are accepted, we will import the package into our work (again, with acknowledgements).

marcusyoung commented 6 years ago

@mshodge Thanks very much for the pull request and interest in the project. Because your changes are on the old R script that I originally wrote for my research, I won't accept the pull request. We now have functions being written within the R package structure. There should be a commit fairly soon with a range of those functions (including, for example, one that also returns route geometry). What I will do is review the work you have done and incorporate your suggestions into the relevant functions as appropriate. I have no problem with you using my original code in the meantime. Once again, thanks for the useful input.