Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Misleading wording in -compute_density #83

Closed
Eskapp opened this issue Dec 3, 2019 · 1 comment
Closed

Misleading wording in -compute_density #83

Eskapp opened this issue Dec 3, 2019 · 1 comment

Comments

@Eskapp
Copy link

Eskapp commented Dec 3, 2019

When using lasinfo with argument -cd, the wording about the unit used is misleading.
Indeed, it seems that when the horizontal unit is unknown, the density is said to be in points per square unit. (This line in the code: https://github.com/LAStools/LAStools/blob/master/src/lasinfo.cpp#L4078)

Square unit is actually a unit that is defined as 100 square feet (as defined here: https://en.wikipedia.org/wiki/Square_(unit)).

Could you confirm that the square unit displayed in lasinfo does not correspond to the official square unit definition? Also, maybe a rewording could be done to avoid confusion.

@rapidlasso
Copy link
Contributor

rapidlasso commented Dec 3, 2019

I did not know such an alternate definition for "square" existed. I simply use the english word "square" in its meaning to "square" things such as "square feet" or "square meter" or "square kilometer" or - if the unit is now known then "square unit" for whatever the unit may be. I assume that most of my users will not confuse this as the other meaning of the term "square" is rather fringe, meaning not common in the LiDAR and pointcloud world.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants