r-spatial / rgee

Google Earth Engine for R
https://r-spatial.github.io/rgee/
Other
677 stars 146 forks source link

Alternative proposal to handle maxFeatures<5000, see issue #229 #234

Closed MatthieuStigler closed 2 years ago

MatthieuStigler commented 2 years ago

This is regarding issue #229, suggesting to simply recompute the size of the collection as soon as the user changes the maxFeatures parameter (i.e. doesn't leave it to 5000). This seems to be more in line with the documentation, and avoid silently changing back the parameter to 5000.

What do you think?

csaybar commented 2 years ago

Ahhh this is much much better thank you so much @MatthieuStigler! :+1: :)