Home - News ( United States | United Kingdom | Italy | Germany ) - Football scores

Showing content from https://github.com/jessecambon/tidygeocoder/issues/71 below:

Add support for Mapbox service · Issue #71 · jessecambon/tidygeocoder · GitHub

Following #62 , I have implemented Mapbox rebased on reverse-geocoding branch.

• Docs: https://docs.mapbox.com/api/search/geocoding/

• Playground: https://docs.mapbox.com/playground/

See the checklist below:

• data-raw/api_reference.R updated
• Modify R/data.R

• Added sandbox/query_debugging/mapbox_test.R
• Added sandbox/reverse_queries/mapbox_reverse.R

• R/utils.R - Utils added for Mapbox
• R/query_factory.R updated
• R/geo_methods.R updated
• R/geo.R updated
• R/reverse_geo.R updated

• testthat test added
• Modify regression_testing.Rmd
• Run tests included in https://github.com/jessecambon/tidygeocoder/blob/master/cran-comments.md

Some Notes not related with the PR itself:

#> * checking CRAN incoming feasibility ... NOTE
#> Version contains large components (1.0.2.9000)

And this:

Found the following (possibly) invalid URLs:
  URL: https://osm.org/copyright (moved to https://www.openstreetmap.org/copyright)
    From: README.md
    Status: 200
    Message: OK

I checked and that url is generated by the response of the reverse_geocode on the README (see table on https://github.com/dieghernan/tidygeocoder/blob/master/README.md#usage), so not related with the package.

• Update README

There are two additional topics about which I would like to gather your feedback,

Github Actions can run R CMD CHECK on a package repo over different platforms and R versions. I use this on my packages, you can see more on https://github.com/r-lib/actions/blob/master/examples/README.md

I included my version while developing. This Github Action would run regularly once a week and after every commit/PR on branchs master or main,, checks are performed on devel, release, oldrel on Linux, macOS and Windows (as CRAN):
https://github.com/dieghernan/tidygeocoder/actions/runs/620783144

Before removing it I just wanted to check if you would like to integrate this on your repo.

This is possible on Mapbox but under the commercial endpoint mapbox.places-permanent

https://docs.mapbox.com/api/search/geocoding/#batch-geocoding

I do not have that kind of access, so I didn't include it. I asked Mapbox Team if there is any way I can get a limited dev account for checking, I would let you know.

What I can do if this is strictly required is to include the capability using a mock answer of the API as example (see here), but I won't be able to check it if I don't get a permanent API key (maybe someone with that kind of key could help, @maxachis¿?).

Regards

RetroSearch is an open source project built by @garambo | Open a GitHub Issue

Search and Browse the WWW like it's 1997 | Search results from DuckDuckGo

HTML: 3.2 | Encoding: UTF-8 | Version: 0.7.4