Skip to content

feat: Add fix LAZ headers workflow TDE-1649 - #1080

Merged
schmidtnz merged 11 commits into
masterfrom
feat/add-pdal-workflow-TDE-1649
Oct 30, 2025
Merged

feat: Add fix LAZ headers workflow TDE-1649#1080
schmidtnz merged 11 commits into
masterfrom
feat/add-pdal-workflow-TDE-1649

Conversation

@schmidtnz

Copy link
Copy Markdown
Contributor

⚠️ 🚧 Merge after associated topo-imagery PR linz/topo-imagery#1419 🚧 ⚠️

Motivation

As a Elevation Data Manager
I want to fix the header information across tens of thousands of LAZ files
so that these files can be uploaded to the LINZ Data Service

Modifications

New workflow to run the new topo-imagery pdal scripts.

Verification

Manual workflow runs.

@schmidtnz
schmidtnz requested review from a team as code owners October 3, 2025 01:29
@schmidtnz
schmidtnz requested review from MDavidson17, amfage, blacha and paulfouquet and removed request for MDavidson17 October 3, 2025 01:29

@paulfouquet paulfouquet left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Similar comment to this one, what are your thoughts about giving more flexibility on the pdal / pdal translate command parameters to be used?

@schmidtnz
schmidtnz force-pushed the feat/add-pdal-workflow-TDE-1649 branch from f2b09c8 to 47e1f60 Compare October 13, 2025 01:02
@schmidtnz

Copy link
Copy Markdown
Contributor Author

Similar comment to this one, what are your thoughts about giving more flexibility on the pdal / pdal translate command parameters to be used?

I think it would be a great addition in a future version and slightly beyond scope at the moment.
A similar workflow can be added to implement custom gdal commands as well.

Comment thread workflows/pointcloud/fix_laz_header.yaml Outdated
Co-authored-by: paulfouquet <86932794+paulfouquet@users.noreply.github.com>
@schmidtnz
schmidtnz requested a review from paulfouquet October 13, 2025 19:44
@paulfouquet paulfouquet changed the title feat: Add pdal workflow TDE-1649 feat: Add fix LAZ headers workflow TDE-1649 Oct 13, 2025
Comment thread workflows/pointcloud/fix_laz_header.yaml Outdated
Comment thread workflows/pointcloud/fix_laz_header.yaml Outdated
Comment thread workflows/pointcloud/fix_laz_header.yaml Outdated
amfage
amfage previously approved these changes Oct 13, 2025
Comment thread workflows/pointcloud/fix_laz_header.yaml Outdated
Comment thread workflows/pointcloud/fix_laz_header.yaml
@schmidtnz
schmidtnz requested a review from amfage October 28, 2025 01:31
@schmidtnz
schmidtnz added this pull request to the merge queue Oct 30, 2025
Merged via the queue into master with commit f2ad314 Oct 30, 2025
3 checks passed
@schmidtnz
schmidtnz deleted the feat/add-pdal-workflow-TDE-1649 branch October 30, 2025 01:32
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

Successfully merging this pull request may close these issues.

3 participants