Skip to content

feat: data-prep script improvements BM-1571#98

Merged
ccbblin merged 11 commits intomasterfrom
feat/data-prep-script-improvements
Mar 30, 2026
Merged

feat: data-prep script improvements BM-1571#98
ccbblin merged 11 commits intomasterfrom
feat/data-prep-script-improvements

Conversation

@ccbblin
Copy link
Copy Markdown
Contributor

@ccbblin ccbblin commented Mar 23, 2026

Motivation

script improvements

Modifications

  1. use a proper argument parser
  2. split the parquet writer into its own folder as all of these should have similar arguments
  3. plit the gpd.overlay into N threads as its currently just running on a single cpu

related: linz/topo-workflows#1165

Verification

https://argo.linzaccess.com/workflows/argo/test-topographic-data-preparation-qdhc5?tab=workflow&uid=2f402e35-ae9e-4ab7-8771-ab0aba1d86d8

@ccbblin ccbblin changed the title feat: data-prep script improvements feat: data-prep script improvements BM-1571 Mar 23, 2026
@github-actions
Copy link
Copy Markdown

Changes Summary

Total Features Changed: 0
Datasets Affected: 0
Git Diff Lines: 1

Kart Diff

Kart Diff (0 features)

Git Diff

Git Diff (1 lines)

@ccbblin ccbblin added the container build pull request container for testing label Mar 24, 2026
@ccbblin ccbblin added container build pull request container for testing and removed container build pull request container for testing labels Mar 26, 2026
@ccbblin ccbblin added container build pull request container for testing and removed container build pull request container for testing labels Mar 26, 2026
@ccbblin ccbblin added container build pull request container for testing and removed container build pull request container for testing labels Mar 29, 2026
tawera-manaena
tawera-manaena previously approved these changes Mar 29, 2026
@ccbblin ccbblin added container build pull request container for testing and removed container build pull request container for testing labels Mar 30, 2026
@ccbblin ccbblin requested a review from schmidtnz March 30, 2026 20:14
@ccbblin ccbblin added this pull request to the merge queue Mar 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

container build pull request container for testing

Development

Successfully merging this pull request may close these issues.

4 participants