search

obrl-soil / h3jsr

Access Uber's h3-js library via R and V8 - https://uber.github.io/h3/#/documentation/overview/use-cases
https://obrl-soil.github.io/h3jsr/
Other
67 stars 5 forks source link

CRAN plans / help wanted? #1

Closed MilesMcBain closed 5 years ago

MilesMcBain commented 5 years ago

Are there any significant blockers stopping this from going to CRAN?

We're using this a lot at work, I'd be happy to pitch in to get it there.

obrl-soil commented 5 years ago

Oh! Nothing major afaik, but I was thinking an rcpp-powered version that hooks into the original H3 lib would be more efficient. I saw a couple of people who looked like they were working on that (https://github.com/crazycapivara/h3-r and https://github.com/scottmmjackson/h3r) so I figured this would become redundant quickly, but neither project seems to have progressed much.

MilesMcBain commented 5 years ago

Yeah AFAIK there were difficulties with building the C version on Windows and that has been a blocker for those projects. I think that is just getting better now.

I see the JS wrapper as valuable in its own right, even if a fully blown C one eventually comes along.

obrl-soil commented 5 years ago

Hmm ok, well I guess I could give it a shot. I've just updated the core lib to the current version, so the only blocker now is that the vignette won't build on linux for Mystery Reasons. I'll start an issue for that and would appreciate any insights :)

rafapereirabr commented 4 years ago

Hi all. Any update on getting this package on CRAN?

obrl-soil commented 4 years ago

Life stuff's been in the way, mostly, but funnily enough I was thinking about it the other day. Might do some maintenance tonight and review it against CRAN policies.

rafapereirabr commented 4 years ago

hi @obrl-soil . This is a really great package and it would be a great contribution to the R spatial community. thanks a lot, and take your time !

obrl-soil commented 4 years ago

update: Still intending to submit this one, but there's a major API revamp underway in the core library so I'm just going to wait until that propagates out through the transpile.