Skip to content

Commit

Permalink
Remove node_pool_name arguments in examples (#350)
Browse files Browse the repository at this point in the history
Our current example pipelines don't work anymore out of the box since
#327 requires `node_pool_label`
and `node_pool_name` to either both be set, or neither.

This PR removes the `node_pool_name` since it is very specific to our
own kfp cluster anyway.
  • Loading branch information
RobbeSneyders authored Aug 10, 2023
1 parent 5e5957d commit a2edbe4
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 4 deletions.
2 changes: 0 additions & 2 deletions examples/pipelines/controlnet-interior-design/pipeline.py
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,6 @@
"max_new_tokens": 50,
},
number_of_gpus=1,
node_pool_name="model-inference-pool",
)
segment_images_op = ComponentOp.from_registry(
name="segment_images",
Expand All @@ -56,7 +55,6 @@
"batch_size": 2,
},
number_of_gpus=1,
node_pool_name="model-inference-pool",
)

write_to_hub_controlnet = ComponentOp(
Expand Down
2 changes: 0 additions & 2 deletions examples/pipelines/finetune_stable_diffusion/pipeline.py
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,6 @@
"max_new_tokens": 50,
},
number_of_gpus=1,
node_pool_name="model-inference-pool",
)

write_to_hub = ComponentOp(
Expand All @@ -80,7 +79,6 @@
"image_column_names": ["images_data"],
},
number_of_gpus=1,
node_pool_name="model-inference-pool",
)

pipeline = Pipeline(pipeline_name=pipeline_name, base_path=PipelineConfigs.BASE_PATH)
Expand Down

0 comments on commit a2edbe4

Please sign in to comment.