Skip to content
New issue

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

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

Already on GitHub? # to your account

[3D] Point cloud editing paintbrush tool #60386

Merged
merged 13 commits into from
Mar 6, 2025

Conversation

Withalion
Copy link
Contributor

@Withalion Withalion commented Jan 31, 2025

Description

This PR adds new tool for editing point clouds. The new tool supports free movement, size change, highlighting of already selected area. Users can change the size of selection area by scrolling and the tool starts selecting on left click + drag. Cursor also shows loading while the point cloud is edited.

Screencast_20250131_193039.mp4

Funded by: Klimadatastyrelsen, Septima

@github-actions github-actions bot added this to the 3.42.0 milestone Jan 31, 2025
@Withalion
Copy link
Contributor Author

This PR builds on work done in #60378

Copy link

github-actions bot commented Jan 31, 2025

🪟 Windows builds

Download Windows builds of this PR for testing.
Debug symbols for this build are available here.
(Built from commit 61435a9)

🪟 Windows Qt6 builds

Download Windows Qt6 builds of this PR for testing.
(Built from commit 61435a9)

@nyalldawson nyalldawson added the Frozen Feature freeze - Do not merge! label Jan 31, 2025
@saberraz
Copy link
Contributor

saberraz commented Feb 3, 2025

I have tried this tool. Very nice addition. Some usability issues:

  • Changing paintbrush size is controlled by the middle mouse key (wheel), by rolling back and forth. This is always dedicated for map zooming in/out and it is confusing. It is best if you combine it with a hot-key.
  • More the above point, it would be good to have a dedicated pixel size for the radius of the paintbrush.
  • While using the tool, I pressed the Escape key and it crashed QGIS.

@Withalion
Copy link
Contributor Author

Paintbrush tool now supports cancellation of selection while still dragging by pressing ESC and the size of paintbrush changes by CTRL + mouse wheel.

@Withalion Withalion force-pushed the point-cloud-paintbrush branch from e04fc42 to 2390c81 Compare February 18, 2025 11:56
@nyalldawson nyalldawson removed the Frozen Feature freeze - Do not merge! label Feb 21, 2025
@Withalion Withalion force-pushed the point-cloud-paintbrush branch 2 times, most recently from 1ea0a73 to c820483 Compare February 25, 2025 13:16
@Withalion Withalion force-pushed the point-cloud-paintbrush branch from c820483 to 8b8c856 Compare February 28, 2025 00:43
Crash occurs when edit tool is active and 3D window is closed
Copy link
Member

@wonder-sk wonder-sk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good stuff 🎉 I have left a bunch of comments, but we're on the right track!

@Withalion Withalion force-pushed the point-cloud-paintbrush branch from 2502519 to e8e2141 Compare March 5, 2025 11:39
@Withalion Withalion force-pushed the point-cloud-paintbrush branch from e8e2141 to d92cd12 Compare March 5, 2025 14:38
@Withalion Withalion force-pushed the point-cloud-paintbrush branch from d92cd12 to 61435a9 Compare March 5, 2025 16:25
@wonder-sk wonder-sk merged commit c5b74ed into qgis:master Mar 6, 2025
32 checks passed
@Withalion Withalion deleted the point-cloud-paintbrush branch March 7, 2025 09:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants