r-hyperspec / hyperSpec

hyperSpec: Tools for Spectroscopy (R package)
https://r-hyperspec.github.io/hyperSpec/
GNU General Public License v3.0
12 stars 3 forks source link

Use `sample(index = TRUE)` instead of `isample()` #17

Closed GegznaV closed 3 years ago

GegznaV commented 3 years ago

Instead of isample() use sample(index = TRUE):

The original discussion is in:

GegznaV commented 3 years ago

@cbeleites, @bryanhanson, is it a problem that in base R there is no argument index in sample()? I.e., the signature does not have .... Wouldn't sample(index = TRUE) result in some kind of R CMD check failures?

image

GegznaV commented 3 years ago

OK, here is additional arg drop: image So no problems should arise.