-
Notifications
You must be signed in to change notification settings - Fork 9
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
Adds functionality to find stations in a geographic area #71
Conversation
Finally, I think it's passing all checks and the vignettes are rebuilt. This is ready for review and merging. |
I've spent some time with the vignettes in this PR as well, it's a bit messy with major additions and these edits but everything lines up and these additions necessitated changes to the vignettes, so it works out. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @adamhsparks! Great addition with find_stations_in()
and additional (tons of it!) work with the tests, re-doc and re-building of the vignettes. I have run the checks and everything works as expected.
Adds a new function,
find_stations_in()
, which can take a bounding box or a geospatial data set of polygon(s) and return stations either that fall within those boundaries or the station nearest the centroid(s).This adds two new imports, {sf} (already installed with the {stars} import), and {osmdata} to derive a bounding box for a named place.
I've added full documentation including updating the vingettes and tests.
The vingettees have not yet been rebuilt due to this bug in {curl} jeroen/curl#338 (comment), but functionally the package is ready to go, I think.