Jakub comments

Results 477 comments of


                                            Jakub

Allow classes for window_lsm()

I would suggest the second option (RasterStack) or, if possible both options. (We should also wait for the email reply before committing to any solutions...)

Upgrading internals

I would need to look deeper in the code, but now I am thinking about: 1. Returning terra SpatRaster instead of raster objects 2. Not using raster functions - I...

Hi @mhesselbarth, I can think of three possible approaches for example data reading: ```r # remotes::install_github("nowosad/test") # 1 # Depends: terra library(test) test:::landscape plot(test:::landscape) landscape = test:::landscape # 2 #...

Upgrading internals

- Input - yes, completely agree -- we just can do that invisibly to the users - Output - yep, I am fine with the warnings

Upgrading internals

Awesome. If I is possible, please also consider the point from https://github.com/r-spatialecology/landscapemetrics/issues/224#issuecomment-1024424440.

Possible errors in nLSI and in the documentation

@thellmundt great investigative work! Thank you. I will wait for @mhesselbarth to confirm this issue and proposed solution. Btw - could you share your example raster with us?

Possible errors in nLSI and in the documentation

@thellmundt have you tried to restart R before installing the new version?

Possible errors in nLSI and in the documentation

Hi @mhesselbarth. I am not entirely sure that I understand all of the terminologies you've used. What are "the background cells"? Regarding the second question - let's say that we...

`scale_sample()`: allow irregular buffer size vector and support `sf` as `y` parameter

Hi @mhesselbarth -- what do you think? Looks fine by me.

`scale_sample()`: allow irregular buffer size vector and support `sf` as `y` parameter

@bniebuhr can you prepare a PR?