getindata / kedro-kubeflow

Kedro Plugin to support running workflows on Kubeflow Pipelines
https://kedro-kubeflow.readthedocs.io
Apache License 2.0
46 stars 21 forks source link

Added support for kedro namespaces for parameters #223

Closed DmitriyLamzin closed 1 year ago

DmitriyLamzin commented 1 year ago

Now params which defined as outer_namespace.inner_namespace.param_name are converted to the following structure before adding to the pipeline:

{
  "outer_namespace": {
      "inner_namespace": {
        "param_name": "value",
        ...
      },
      "another_inner_namespace": {
        "param_name": "value"
        ...
      }
  }
}

Description

describe the purpose of the change here

Resolves https://github.com/getindata/kedro-kubeflow/issues/219

PR Checklist
szczeles commented 1 year ago

Looks great, @DmitriyLamzin! Thanks for your contribution

DmitriyLamzin commented 1 year ago

Looks like CI is broken. I don't think my changes could break it.

szczeles commented 1 year ago

@DmitriyLamzin You're right, the CI is broken, I have the fix ready, and we will merge it on Monday.

szczeles commented 1 year ago

@DmitriyLamzin Released as 0.7.4 :tada: