Export pointcloud layers

[uclaros]

Description

This PR allows the users to export a (potentially filtered) point cloud layer to vector or point cloud formats. Current supported formats are:

• Temporary scratch layer (memory layer)
• GeoPackage
• Shapefile
• DXF
• LAZ (requires build WITH_PDAL)

Supports filtering by extent, filtering by Z range, crs re-projection and limiting the number of exported points.

Funded by: State of Vorarlberg and Swiss QGIS user group

[DelazJ]

Suggested change

	<string>Limit total number of exported points to:</string>
	<string>Limit total number of exported points to</string>

No colon on labels.
And unless there is a plan to add more entries, I was wondering whether we need that group box with single item (with almost the same label). Not HIG compliant. What about:

• a single spinbox showing no limit and then people enter their value
  

• a checkbox followed by a spinbox, on the same line
  

We would probably then need to move that single option somewhere upper...

[uclaros]

There's a though to add a separate widget in that group that would limit the total number of points by stopping at a particular node level, resulting in a (kind of) predefined point density.

While groupboxes with single items do clutter the ui, imho in this specific case it makes sense, since it's on a list of collapsible-checkable groupboxes. Having one type of filter without a groupbox would make it look weirder and also make it stand-out like it's a more important setting.

Since the HIG says to avoid using those, I'd propose to keep it like that, if you do not insist!

[DelazJ]

There's a though to add a separate widget in that group that...

The you hit my "unless there is a plan to add more entries"

Having one type of filter without a groupbox would make it look weirder and also make it stand-out like it's a more important setting.

Agreed. Reason why I was suggesting to move it somewhere else (no idea which other options it could have cohabited with though)

Since the HIG says to avoid using those, I'd propose to keep it like that, if you do not insist!

No insistence. See above.

[DelazJ]

@uclaros can you also hide the scrollbar frame border and reduce the margins of the dialog? See #49568

[DelazJ]

And question: should we consider in the future adding "Temporary scratch layer" to vector layer export types also?

[github-actions]

@uclaros
This pull request has been tagged as requiring documentation.

A documentation ticket will be opened at https://github.com/qgis/QGIS-Documentation when this PR is merged.

Please update the description (not the comments) with helpful description and screenshot to help the work from documentors.
Also, any commit having [needs-doc] or [Needs Documentation] in will see its message pushed to the issue, so please be as verbose as you can.

Thank you!

[uclaros]

@uclaros can you also hide the scrollbar frame border and reduce the margins of the dialog? See #49568

I'm not sure I completely agree with that change; it doesn't look too good when the scrollbar appears... I left a comment on your PR

And question: should we consider in the future adding "Temporary scratch layer" to vector layer export types also?

This is not a bad idea! Copy and paste as a temporary scratch layer is not as straight forward as export! Doing it with lots of data would also be faster by skipping the clipboard middleman!

[wonder-sk]

🎉

[github-actions]

@uclaros
A documentation ticket has been opened at qgis/QGIS-Documentation#7745
It is your responsibility to visit this ticket and add as much detail as possible for the documentation team to correctly document this change.
Thank you!