spplot does not handle raster of 1 cell.

[alaindanet]

Hi,
thank you very much for the package!

I would expect that sp::spplot() can plot a raster of size 1 if sp::plot() can do it.
It is not a big issue I guess, but it is annoying when testing/debugging functions with dummy datasets.

library('sp')
r <- raster::raster(nrows = 1, ncols = 1, xmn = 2, xmx = 4, ymn = 26, ymx = 28)
raster::values(r) <- rnorm(length(raster::values(r))) 

# Does not work:
spplot(r)
#> Error in apply(sapply(from@data, is.na), 1, function(x) !all(x)): dim(X) doit avoir un longueur positive
# Work: 
plot(r)

^{Created on 2019-11-10 by the reprex package (v0.3.0)}

[edzer]

Looks like you have to load package raster first.

…

On 10 November 2019 11:05:24 CET, alaindanet ***@***.***> wrote: Hi,> thank you very much for the package!> > I would expect that `sp::spplot()` can plot a raster of size 1 if `sp::plot()` can do it.> It is not a big issue I guess, but it is annoying when testing/debugging functions with dummy datasets.> > ``` r> library('sp')> r <- raster::raster(nrows = 1, ncols = 1, xmn = 2, xmx = 4, ymn = 26, ymx = 28)> raster::values(r) <- rnorm(length(raster::values(r))) > > # Does not work:> spplot(r)> #> Error in ***@***.***, is.na), 1, function(x) !all(x)): dim(X) doit avoir un longueur positive> # Work: > plot(r)> ```> > > > ^{Created on 2019-11-10 by the [reprex package](https://reprex.tidyverse.org) (v0.3.0)}> > -- > You are receiving this because you are subscribed to this thread.> Reply to this email directly or view it on GitHub:> #65

-- Sent from a phone. Please excuse my brevity.

[alaindanet]

Thanks. Unfortunately, it did not solve the problem

library('raster')
#> Le chargement a nécessité le package : sp

r <- raster::raster(nrows = 1, ncols = 1, xmn = 2, xmx = 4, ymn = 26, ymx = 28)
raster::values(r) <- rnorm(length(raster::values(r))) 

# Does not work:
spplot(r)
#> Error in apply(sapply(from@data, is.na), 1, function(x) !all(x)): dim(X) doit avoir un longueur positive
# Work: 
plot(r)

^{Created on 2019-11-10 by the reprex package (v0.3.0)}

[edzer]

Yes, this is a bug. Since it is a corner case and sp is in maintenance mode, I am not going to correct it. Thanks for reporting; PR will be accepted after someone tests it against all reverse dependencies.