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

Squashed pipeline generator #29

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from
Draft

Squashed pipeline generator #29

wants to merge 2 commits into from

Conversation

szczeles
Copy link
Contributor

@szczeles szczeles commented Apr 8, 2022

Description

Provides alternative pipeline generation, suitable to run SparkML pipelines with lazy evaluation of steps and MemoryDataSet artifacts.

PR Checklist

@@ -41,7 +44,7 @@ def __init__(self, config: PluginConfig, project_name, context):
)
self.run_config = config.run_config
self.run_name = self._generate_run_name(config)
self.generator = PipelineGenerator(
self.generator = globals()[self.run_config.generator_class](
Copy link
Contributor

Choose a reason for hiding this comment

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

Please don't do that...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

:smutnazaba:

] = "{{$.inputs.parameters['mlflow_tracking_token']}}"
command.append(
_decorate_command(
f"kedro vertexai -e {self.context.env} mlflow-start --output /tmp/mlflow.run.id {self.run_name} &&",
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we still need this if we're running in a single node?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good question! It has one nice feature of setting the tag with vertex run id, so I decided to keep it. It could be covered by some custom hook in the project, but it requires custom development.
image

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