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

Drag zoomed in LiDAR data in RGL window #442

Closed
Saadi4469 opened this issue Jun 15, 2021 · 8 comments
Closed

Drag zoomed in LiDAR data in RGL window #442

Saadi4469 opened this issue Jun 15, 2021 · 8 comments
Assignees
Labels
Enhancement Not actually a bug but a possible improvement Question Not an issue, just a question

Comments

@Saadi4469
Copy link

Saadi4469 commented Jun 15, 2021

In some softwares the user can hold down the middle mouse button to drag the visual data. How can this be done while correcting the tree tops identified in a zoomed in view?

TTops.manual = find_trees(LAS_denoised, manual(LAS_TTops))

@Saadi4469 Saadi4469 changed the title Drag zoomed in LIDAR data in Plot3D window Drag zoomed in LIDAR data in RGL window Jun 15, 2021
@Saadi4469 Saadi4469 changed the title Drag zoomed in LIDAR data in RGL window Drag zoomed in LiDAR data in RGL window Jun 15, 2021
@Jean-Romain Jean-Romain self-assigned this Jun 15, 2021
@Jean-Romain Jean-Romain added the Enhancement Not actually a bug but a possible improvement label Jun 15, 2021
@Jean-Romain
Copy link
Collaborator

Good question. I did not use this function for years but if I remember well you identify the trees with the right button, and the right button is also associated to the drag event. So, I guess drag action is overwritten by the selection action. The reason is that manual() was existing before the 3D rendering in lidR supports the drag action.

I guess I could put the selection with the middle button to keep the drag action.

@Jean-Romain Jean-Romain added the Question Not an issue, just a question label Jun 15, 2021
@Jean-Romain
Copy link
Collaborator

Jean-Romain commented Jun 15, 2021

Which version of the package are you using ? It seems that the function no longer works... I don't know why.

Jean-Romain added a commit that referenced this issue Jun 15, 2021
@Jean-Romain
Copy link
Collaborator

I fixed the error. I'd like to know which version of lidR and rgl were in use on your computer because the bug I experienced was not existing before and I did not change the function for years. So I suspect an slight modification in rgl itself.

Also the default button for selection is now the middle button. This automatically enabled the right button that is the default for panning in lidR.

Thank for reporting

@Saadi4469
Copy link
Author

I am using the latest version of lidR ver. 3.1.3 and rgl ver. 0.106.8 available on CRAN. And I still have to use the right mouse button to add or flag tree tops, and the middle mouse button stretches the view.

@Jean-Romain
Copy link
Collaborator

Yes of course. You must install the latest version from github. It is not yet on CRAN

@Saadi4469
Copy link
Author

for both lidR and rgl, or just lidR?

@Jean-Romain
Copy link
Collaborator

Just lidR.This is the only source code I can update. I've no control on rgl.

@Saadi4469
Copy link
Author

Yes, the drag function works now thank you and cheers on fixing the error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Not actually a bug but a possible improvement Question Not an issue, just a question
Projects
None yet
Development

No branches or pull requests

2 participants