Skip to content

[com7] remove runSplitInputs.py and integrate functionality into runCom7Regional.py#1181

Merged
fso42 merged 2 commits intomasterfrom
combineCom7Runs
Sep 16, 2025
Merged

[com7] remove runSplitInputs.py and integrate functionality into runCom7Regional.py#1181
fso42 merged 2 commits intomasterfrom
combineCom7Runs

Conversation

@fso42
Copy link
Copy Markdown
Contributor

@fso42 fso42 commented Sep 10, 2025

Needed for QGis Connector scripts

@fso42 fso42 requested a review from awirb September 10, 2025 09:01
@fso42 fso42 self-assigned this Sep 10, 2025
@fso42 fso42 added the enhancement New feature or request label Sep 10, 2025
Copy link
Copy Markdown
Contributor

@awirb awirb left a comment

Choose a reason for hiding this comment

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

Why delete the runSplitInputs script? so always use runCom7Regional and there decide which of the two or if both of the functionalities to use? Actually I agree that make more sense maintenance wise

@fso42 fso42 added this to the Version 2.0 milestone Sep 15, 2025
…onality into `runCom7Regional.py`

- Added `--split-inputs` flag to execute `splitInputs` preprocessing via `runCom7Regional.py`.

refactor(com7Regional): rename `splitInputsFlag` to `splitInputs` for consistency

- Updated parameter, variables, and references for improved clarity and alignment with naming conventions.

refactor(runCom7Regional): alias `splitInputs` import for improved readability
@fso42 fso42 merged commit 914f501 into master Sep 16, 2025
5 checks passed
@fso42 fso42 deleted the combineCom7Runs branch September 16, 2025 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants