Maybe error in entropy and VCI #743
Replies: 3 comments 5 replies
-
I do not see that. Please cite some code and explain what you think is incorrect. I hardly understand what may be incorrect based on your explanation. Thanks |
Beta Was this translation helpful? Give feedback.
-
Yes, the problem is that the script in the function entropy returns value of VCI rather than entropy. Based on the definition, VCI is entropy divided by the maximum possible entropy. |
Beta Was this translation helpful? Give feedback.
-
Another thing may be that, if the given value for VCI is larger than the maximum canopy height, it still returns value, but that value is incorrect, I think it may be better to add an if loop to avoid such situation. |
Beta Was this translation helpful? Give feedback.
-
Hi Jean-Romain,
Thanks for your contribution in developing such good and useful package for computing data metrics from the ALS data.
While I have a question on the script computing entropy and vertical complexity index (VCI). I noticed that the function computing VCI returned the same value of function computing entropy did when you set zmax as the maximum canopy height and set by as 1.
I checked the script in the github and found that the function entropy divided the value by the logrithm value of the maximum canopy height, while this should be in the equation of VCI, not entropy. And in the function VCI, it just returns the value of entropy value without extra computation.
Best regards,
Beta Was this translation helpful? Give feedback.
All reactions