baddstats / polyclip

R package polyclip: a port of the Clipper library for polygon geometry
19 stars 6 forks source link

add PACKAGE = "polyclip" to .Call() calls #1

Closed richierocks closed 7 years ago

richierocks commented 7 years ago

The spatstat package is going to feature prominently in an upcoming DataCamp course. Unfortunately, neither the version on CRAN, nor the development version work correctly on the DataCamp platform. The problem is that when C code is called, sometimes an error occurs regarding locating the call.

ppp(x=runif(10), y=runif(10), window=disc(radius=10))
## Error: "Cmatchxy" not resolved from current namespace (spatstat)

The fix is to include a PACKAGE argument to calls to .C() and .Call(). This fix should be applicable to other platforms too, making spatstat more widely available.

I've also made this change to the dependencies spatstat.utils and polyclip.

baddstats commented 7 years ago

OK, accepted