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

plot extent of las file with mapview #680

Closed
wiesehahn opened this issue May 10, 2023 · 6 comments
Closed

plot extent of las file with mapview #680

wiesehahn opened this issue May 10, 2023 · 6 comments
Assignees
Labels
Feature request Asking for a new feature Question Not an issue, just a question

Comments

@wiesehahn
Copy link

lidR::plot() is able to plot a LAScatalog with mapview (e.g. plot(ctg, mapview=TRUE)).

With single LAS files this it not possible. Is there a reason for this or would it be possible to implement it so that plot(las, mapview=TRUE) will plot the extent of the LAS file on top of some basemap with mapview?

@Jean-Romain
Copy link
Collaborator

plot(header(las), mapview = TRUE)

@Jean-Romain Jean-Romain self-assigned this May 10, 2023
@Jean-Romain Jean-Romain added the Question Not an issue, just a question label May 10, 2023
@wiesehahn
Copy link
Author

thanks, exactly that.
Just seems a little inconsistent, but now I know.

@Jean-Romain
Copy link
Collaborator

plot(las) plots in 3D this is the default behavior expected by users I guess. I could add something like plot(las, 2d = TRUE) but this would also be inconsistant. I'm open to suggestion if you have an idea.

@wiesehahn
Copy link
Author

wiesehahn commented May 10, 2023

I am fine with the current state now that I know how.
But intuitively I expected that plot(x) returns a 3D plot (default and most expected by users) for LAS files and plot(x, mapview=TRUE) returns a 2D plot similar to the case if x is a LAScatalog. In the current state defining mapview=TRUE in the case of LASfiles does not change anything.

@Jean-Romain Jean-Romain added the Feature request Asking for a new feature label May 10, 2023
@Jean-Romain
Copy link
Collaborator

This argument does not exist for plot.LAS. It only exists for plot.LAScatalog but I could add it. That makes sense.

@Jean-Romain
Copy link
Collaborator

It is now possible to use plot(las, mapview = TRUE)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature request Asking for a new feature Question Not an issue, just a question
Projects
None yet
Development

No branches or pull requests

2 participants