Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensure proper spacing in error messages #2399

Merged
merged 7 commits into from
Feb 7, 2024
Merged
Show file tree
Hide file tree
Changes from 5 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions docs/book/toc.md
Original file line number Diff line number Diff line change
Expand Up @@ -94,6 +94,7 @@
* [Tekton Orchestrator](stacks-and-components/component-guide/orchestrators/tekton.md)
* [Airflow Orchestrator](stacks-and-components/component-guide/orchestrators/airflow.md)
* [Skypilot VM Orchestrator](stacks-and-components/component-guide/orchestrators/skypilot-vm.md)
* [HyperAI Orchestrator](stacks-and-components/component-guide/orchestrators/hyperai.md)
* [Develop a custom orchestrator](stacks-and-components/component-guide/orchestrators/custom.md)
* [Artifact Stores](stacks-and-components/component-guide/artifact-stores/artifact-stores.md)
* [Local Artifact Store](stacks-and-components/component-guide/artifact-stores/local.md)
Expand Down
2 changes: 1 addition & 1 deletion src/zenml/cli/stack_components.py
Original file line number Diff line number Diff line change
Expand Up @@ -941,7 +941,7 @@ def register_stack_component_flavor_command(source: str) -> None:
source_root = source_utils.get_source_root()

cli_utils.error(
f"Flavor registration failed! ZenML tried loading the"
f"Flavor registration failed! ZenML tried loading the "
f"module `{source}` from path `{source_root}`. If this is "
"not what you expect, then please ensure you have run "
"`zenml init` at the root of your repository.\n\n"
Expand Down
2 changes: 1 addition & 1 deletion src/zenml/integrations/kserve/steps/kserve_deployer.py
Original file line number Diff line number Diff line change
Expand Up @@ -353,7 +353,7 @@ def kserve_custom_model_deployer_step(
# verify that a custom deployer is defined
if not params.custom_deploy_parameters:
raise ValueError(
"Custom deploy parameter which contains the path of the",
"Custom deploy parameter which contains the path of the ",
"custom predict function is required for custom model deployment.",
)

Expand Down
8 changes: 4 additions & 4 deletions src/zenml/integrations/mlflow/steps/mlflow_deployer.py
Original file line number Diff line number Diff line change
Expand Up @@ -228,10 +228,10 @@ def mlflow_model_registry_deployer_step(
"""
if not registry_model_version and not registry_model_stage:
raise ValueError(
"Either registry_model_version or registry_model_stage must"
"be provided in addition to registry_model_name to the MLflow"
"model registry deployer step. Since the"
"mlflow_model_registry_deployer_step is used in conjunction with"
"Either registry_model_version or registry_model_stage must "
"be provided in addition to registry_model_name to the MLflow "
"model registry deployer step. Since the "
"mlflow_model_registry_deployer_step is used in conjunction with "
"the mlflow_model_registry."
)

Expand Down
10 changes: 5 additions & 5 deletions src/zenml/integrations/seldon/steps/seldon_deployer.py
Original file line number Diff line number Diff line change
Expand Up @@ -378,15 +378,15 @@ def seldon_mlflow_registry_deployer_step(
# Seldon Core model deployer are available
if not registry_model_name:
raise ValueError(
"registry_model_name must be provided to the MLflow"
"registry_model_name must be provided to the MLflow "
"model registry deployer step."
)
elif not registry_model_version and not registry_model_stage:
raise ValueError(
"Either registry_model_version or registry_model_stage must"
"be provided in addition to registry_model_name to the MLflow"
"model registry deployer step. Since the"
"mlflow_model_registry_deployer_step is used in conjunction with"
"Either registry_model_version or registry_model_stage must "
"be provided in addition to registry_model_name to the MLflow "
"model registry deployer step. Since the "
"mlflow_model_registry_deployer_step is used in conjunction with "
"the mlflow_model_registry."
)
if service_config.implementation != "MLFLOW_SERVER":
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,7 @@ def prepare_or_run_pipeline(
)
if deployment.schedule:
logger.warning(
"Skypilot Orchestrator currently does not support the"
"Skypilot Orchestrator currently does not support the "
"use of schedules. The `schedule` will be ignored "
"and the pipeline will be run immediately."
)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,7 @@ def prepare_or_run_pipeline(
"""
if deployment.schedule:
logger.warning(
"Local Docker Orchestrator currently does not support the"
"Local Docker Orchestrator currently does not support the "
"use of schedules. The `schedule` will be ignored "
"and the pipeline will be run immediately."
)
Expand Down