bhaskarvk / tilegramsR

R wrapper for Tilegrams
https://bhaskarvk.github.io/tilegramsR/
Other
52 stars 2 forks source link

Unable to duplicate RPubs example #1

Closed smach closed 8 years ago

smach commented 8 years ago

I was trying to duplicate your code here

http://rpubs.com/bhaskarvk/electoral-Map-2016

But I keep getting this error message

Error in derivePoints(data, lng, lat, missing(lng), missing(lat), "addMarkers") : 
  addMarkers requires non-NULL longitude/latitude values

I had the same problem when trying to create a tilegram using my own data with the 538 style. Any ideas?

bhaskarvk commented 8 years ago

I think it's because of https://github.com/rstudio/leaflet/pull/310. If you git clone rstudio/leaflet and build after merging that PR, that error should go away.

bhaskarvk commented 8 years ago

devtools::install_github("rstudio/leaflet#310") should do it, I think.

smach commented 8 years ago

That did the trick, thanks!

On Sat, Oct 15, 2016 at 10:48 AM, Bhaskar V. Karambelkar < notifications@github.com> wrote:

devtools::install_github("rstudio/leaflet#310") should do it, I think.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/bhaskarvk/tilegramsR/issues/1#issuecomment-253988657, or mute the thread https://github.com/notifications/unsubscribe-auth/ADPKcJ_ylZLXq8cAx1ozqgY-bDRPovhnks5q0OfUgaJpZM4KXuJz .

bhaskarvk commented 8 years ago

thanks for confirming, I will merge that PR soon.

bhaskarvk commented 8 years ago

The fix has now been merged in leaflet master.

smach commented 8 years ago

Great, thanks!