#417

NEWS.md

@@ -4,8 +4,9 @@ If you are viewing this file on CRAN, please check [the latest news on GitHub](h
 
 - Fix: `las_check(..., deep = TRUE)` was not working in parallel ([#411](https://github.com/Jean-Romain/lidR/issues/411).
 - Fix: `readLAScatalog()` throw an more informative error when attempting to read an non-existing folder.
-- Fix: the LAScatalog processing engine printed the outputs twice for rare function that print something like `las_check()`  ([#414](https://github.com/Jean-Romain/lidR/issues/414)
+- Fix: the LAScatalog processing engine printed the outputs twice for rare function that print something like `las_check()`  ([#414](https://github.com/Jean-Romain/lidR/issues/414))
 - New: `las_check()` introduces a new type of message called "message". Some message previously classified as "warning" are now classified as "message". Warnings are now displayed in orange and messages in yellow. The output of `las_check()` has now 3 items instead of 2.
+- Enhance: `max_cr_factor` in `silva2019()` is now allowed to be in [0, inf[ instead of [0,1] (([#417](https://github.com/Jean-Romain/lidR/issues/417))
 
 ## lidR v3.1.2 (Release date: 2021-03-11)
 

R/algorithm-its.R

@@ -333,7 +333,7 @@ silva2016 = function(chm, treetops, max_cr_factor = 0.6, exclusion = 0.3, ID = "
   assert_is_all_of(treetops, "SpatialPointsDataFrame")
   assert_is_a_number(max_cr_factor)
   assert_is_a_number(exclusion)
-  assert_all_are_in_open_range(max_cr_factor, 0, 1)
+  assert_all_are_positive(max_cr_factor)
   assert_all_are_in_open_range(exclusion, 0, 1)
 
   chm            <- lazyeval::uq(chm)