Skip to content

clarifai pipeline run --dev#1012

Open
rizzip wants to merge 4 commits intomasterfrom
feat/pipeline-run-dev
Open

clarifai pipeline run --dev#1012
rizzip wants to merge 4 commits intomasterfrom
feat/pipeline-run-dev

Conversation

@rizzip
Copy link
Copy Markdown
Contributor

@rizzip rizzip commented Apr 8, 2026

Why

  • Allow simpler testing for pipeline development, isolated from mainline pipeline objects but leveraging full orchestration rather than local execution

How

  • Added --dev flag to clarifai pipeline run to upload local code to an ephemeral {pipeline_id}-dev pipeline before running; only changed steps are re-uploaded using hash-based change detection
  • Added optional positional PATH argument to clarifai pipeline run for specifying a pipeline directory or config file (prefers config-lock.yaml, falls back to config.yaml)
  • Cleaned up logs for clarifai pipeline run

Tests

Notes

rizzip added 2 commits April 8, 2026 11:31
* Added `--dev` flag to `clarifai pipeline run` to upload local code to an ephemeral `{pipeline_id}-dev` pipeline before running; only changed steps are re-uploaded using hash-based change detection
* Added optional positional `PATH` argument to `clarifai pipeline run` for specifying a pipeline directory or config file (prefers `config-lock.yaml`, falls back to `config.yaml`)
@rizzip rizzip marked this pull request as ready for review April 8, 2026 19:22
@rizzip rizzip requested a review from ackizilkale April 9, 2026 13:42
@github-actions
Copy link
Copy Markdown

Code Coverage

Package Line Rate Health
clarifai 45%
clarifai.cli 63%
clarifai.cli.templates 67%
clarifai.cli.templates.toolkits 100%
clarifai.client 65%
clarifai.client.auth 67%
clarifai.constants 100%
clarifai.datasets 100%
clarifai.datasets.export 69%
clarifai.datasets.upload 75%
clarifai.datasets.upload.loaders 37%
clarifai.models 100%
clarifai.rag 0%
clarifai.runners 52%
clarifai.runners.models 60%
clarifai.runners.pipeline_steps 39%
clarifai.runners.pipelines 65%
clarifai.runners.utils 62%
clarifai.runners.utils.data_types 72%
clarifai.schema 100%
clarifai.urls 58%
clarifai.utils 64%
clarifai.utils.evaluation 16%
clarifai.workflows 95%
Summary 60% (12066 / 19978)

Minimum allowed line rate is 50%

Comment thread clarifai/cli/pipeline.py
monitor_interval=monitor_interval,
input_args_override=input_args_override,
)
click.echo(json.dumps(result, indent=2, default=str))
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.

did you remove this intentionally?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, the json was polluting the output imo.

Copy link
Copy Markdown
Contributor

@ackizilkale ackizilkale left a comment

Choose a reason for hiding this comment

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

lgtm. I was just curious about one log you removed. If not important all good.

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.

2 participants