r-spatialecology/landscapemetrics

show_patches

Closed this issue · 8 comments

Nowosad commented

Shouldn't show_patches() be a part of the landscapetools package? What do you think?

marcosci commented

If Max is fine with that, it probably makes sense :)

mhesselbarth commented

I'm fine with that, sure

marcosci commented

show_patches is now in landscapetools

  • 👍1
marcosci commented

Max just pointed out that it also makes sense to do the same with cclabel and pad_raster (or make pad_raster just an internal function, there is probably no actual use case for that).

Everyone fine with that?

Nowosad commented

Can we wait a few weeks for a decision about it? I am going to add a piece of rcpp code in the (near) future and we can decide what do to with all of it;)

marcosci commented

Sure thing :)

marcosci commented

Max and I piggybacked show_patches back into the package and Max also included show_correlation.
We think that it is at least a good point for the planned publication, but also in the context of including some functions to select metrics and/or look at the results.

Fine with you @Nowosad ? Completely open for arguments against that.

Nowosad commented

I do not have a strong opinion here. Will let you know if I would have one;)

  • 👍1