Conversation
Agent-Logs-Url: https://github.com/maxnutz/pypsa_validation_processing/sessions/0820f2d0-3c71-48d8-bb2f-40ef604c013d Co-authored-by: maxnutz <81740567+maxnutz@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Add local workflow testing pixi task
Add Apr 23, 2026
workflow_test Pixi task for full local workflow/config matrix validation
Contributor
Reviewer's guide (collapsed on small PRs)Reviewer's GuideAdds a new Pixi task Sequence diagram for Pixi workflow_test task executionsequenceDiagram
actor Developer
participant Pixi
participant workflow_py
participant pytest_cli
Developer->>Pixi: pixi run workflow_test
Pixi->>workflow_py: python workflow.py --config pypsa_validation_processing/configs/config.country-timeseries.yaml
workflow_py-->>Pixi: exit code
Pixi->>workflow_py: python workflow.py --config pypsa_validation_processing/configs/config.country-year.yaml
workflow_py-->>Pixi: exit code
Pixi->>workflow_py: python workflow.py --config pypsa_validation_processing/configs/config.region-timeseries.yaml
workflow_py-->>Pixi: exit code
Pixi->>workflow_py: python workflow.py --config pypsa_validation_processing/configs/config.region-year.yaml
workflow_py-->>Pixi: exit code
Pixi->>pytest_cli: pytest tests/ -v
pytest_cli-->>Pixi: test results
Pixi-->>Developer: aggregated command results
Flow diagram for workflow_test Pixi task command chainflowchart LR
dev[Developer runs pixi run workflow_test]
pixi[Pixi task workflow_test]
cts[Run workflow.py with config.country-timeseries.yaml]
cy[Run workflow.py with config.country-year.yaml]
rts[Run workflow.py with config.region-timeseries.yaml]
ry[Run workflow.py with config.region-year.yaml]
tests[Run pytest tests/ -v]
dev --> pixi
pixi --> cts
cts --> cy
cy --> rts
rts --> ry
ry --> tests
File-Level Changes
Assessment against linked issues
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Contributor
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- The
workflow_testtask command is quite long and hard to scan; consider factoring the repeatedpython workflow.py --config ...invocations into a small wrapper script or separate Pixi task(s) so the config matrix is easier to maintain and extend. - The test currently depends on splitting the task string by the literal delimiter
" && "; if Pixi’s task syntax changes or additional shell flags are needed this will become brittle, so you might instead assert the full string or introduce a simple parser/helper that encapsulates the command structure.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The `workflow_test` task command is quite long and hard to scan; consider factoring the repeated `python workflow.py --config ...` invocations into a small wrapper script or separate Pixi task(s) so the config matrix is easier to maintain and extend.
- The test currently depends on splitting the task string by the literal delimiter `" && "`; if Pixi’s task syntax changes or additional shell flags are needed this will become brittle, so you might instead assert the full string or introduce a simple parser/helper that encapsulates the command structure.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This adds a local Pixi task to execute the full workflow/config matrix plus tests in one command, so contributors can quickly verify compatibility across all supported temporal and regional scalings.
Pixi task: full sequential workflow check
workflow_testunder[tasks]inpixi.toml.Coverage of required config combinations
config.country-timeseries.yamlconfig.country-year.yamlconfig.region-timeseries.yamlconfig.region-year.yamlpytest tests/ -vRegression guard for task definition
tests/test_pixi_tasks.pyto assertworkflow_testcontains the exact required command list in order.Summary by Sourcery
Add a Pixi task to run the full workflow configuration matrix followed by tests in a single command and guard it with a regression test.
New Features:
workflow_testPixi task that sequentially runs all supported workflow configurations and the test suite.Tests:
workflow_testPixi task runs the expected commands in the correct order.