mark padgham

Results 147 issues of mark padgham

See #77 for related stuff

future stuff

@Mashin6 raised valid concerns in #252, supported by @stragu, that the current design in counter-intuitive: **OR** ``` r add_osm_features (features = c (...)) ``` **AND** ``` r add_osm_feature() |> add_osm_feature()...

enhancement
help wanted

Because I finally had call to trim a huge data set (New York City within the official boundary polygon => around 700,000 vertices submitted to `sp::point.in.polygon` or `sf::st_within`). The latter...

future stuff

Coz things have moved on a lot since [these comments](https://github.com/hrbrmstr/overpass/issues/13#event-1623082590). It should be pretty straightforward to `Rcpp::RcppParallel`-ize the main loops over nodes, ways, relations in `src/osmdata_sf/sp`.

future stuff

Instead of repeatedly bombing the `overpass` API, it'd be pretty easy to implement a local caching system that would record the call and store the pre-processed data returned from the...

enhancement

Thanks to @Robinlovelace, and extending from [this discussion](https://github.com/cyipt/cyipt/issues/22), the rules provided [here](https://github.com/keepright/keepright/tree/master/checks) could easily be translated into `osmdata` terms and used to analyse local data in same way. An `osmdata_check()`...

future stuff

These are [permitted in OSM](https://wiki.openstreetmap.org/wiki/Relation:multipolygon#Touching_inner_rings) yet not by SF. [`osmdata.cpp`](https://github.com/osmdatar/osmdata/blob/master/src/osmdata.cpp) needs to be modified to merge any touching inner rings into single rings. See also [this extended OSM discussion](https://wiki.openstreetmap.org/wiki/Talk:Relation:multipolygon#Touching_inner_rings).

coz it oughta be done

This would probably be quite useful, and easily added to `opq ()`. Should probably also default to recurse down **relation** (`>>`).

coz it oughta be done

This PR is one way to resolve an error that arises when following @jimhester's advice on ["Getting started with cpp11"](https://cpp11.r-lib.org/articles/cpp11.html#using-cpp11-in-a-package), which simply says to add a "LinkingTo" statement. `cpp11` has...

See [this issue in separate repo](https://github.com/ropensci/osmdata/issues/157) for a description. The current implementation is too limited to be of wide practical use because python bottle uses a rather restricted default upper...