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.

Sign up for GitHub

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

Jump to bottom

[sdk] no module named `kfp.components.types' error when a deprecated package is used. #10282

Closed
hsinhoyeh opened this issue Dec 4, 2023 · 0 comments · Fixed by #10283
Closed

[sdk] no module named `kfp.components.types' error when a deprecated package is used. #10282

hsinhoyeh opened this issue Dec 4, 2023 · 0 comments · Fixed by #10283
Labels
area/sdk kind/bug

Comments

@hsinhoyeh
Copy link
Contributor

Environment

  • KFP version: 2.0.4, as well as on top of master

  • KFP SDK version: sdk-v2.4.0, the latest sdk version as well as on top of master branch

Steps to reproduce

build pipeline with a deprecated package encounter error.

  File "/usr/local/lib/python3.11/dist-packages/kfp/deprecated/dsl/_component_bridge.py", line 29, in <module>
    from kfp.deprecated.dsl import component_spec as dsl_component_spec
  File "/usr/local/lib/python3.11/dist-packages/kfp/deprecated/dsl/component_spec.py", line 21, in <module>
    from kfp.deprecated.dsl import type_utils
  File "/usr/local/lib/python3.11/dist-packages/kfp/deprecated/dsl/type_utils.py", line 19, in <module>
    from kfp.components.types import type_utils
ModuleNotFoundError: No module named 'kfp.components.types'

the module kfp.components.types is moved to kfp.dsl.types during #9710.

Expected result

build without error

Materials and Reference

Impacted by this bug? Give it a 👍.
hsinhoyeh added a commit to hsinhoyeh/pipelines that referenced this issue Dec 4, 2023
@hsinhoyeh
use kfp.dsl.types to replace kfp.components.types
80d92f4 
Fixes: kubeflow#10282
Signed-off-by: hsinhoyeh <yhh92u@gmail.com>
@hsinhoyeh hsinhoyeh mentioned this issue Dec 4, 2023
Merged
@Tomcli Tomcli mentioned this issue Dec 5, 2023
Merged
1 task
connor-mccarthy pushed a commit that referenced this issue Dec 13, 2023
@hsinhoyeh
fix(sdk): use kfp.dsl.types to replace kfp.components.types Fixes #10282
b40912c 
 (#10283)

* use kfp.dsl.types to replace kfp.components.types

Fixes: #10282
Signed-off-by: hsinhoyeh <yhh92u@gmail.com>

* fix import artifact_types

---------

Signed-off-by: hsinhoyeh <yhh92u@gmail.com>
stijntratsaertit pushed a commit to stijntratsaertit/kfp that referenced this issue Feb 16, 2024
@hsinhoyeh @stijntratsaertit
fix(sdk): use kfp.dsl.types to replace kfp.components.types Fixes kub…
b110742 
…eflow#10282 (kubeflow#10283)

* use kfp.dsl.types to replace kfp.components.types

Fixes: kubeflow#10282
Signed-off-by: hsinhoyeh <yhh92u@gmail.com>

* fix import artifact_types

---------

Signed-off-by: hsinhoyeh <yhh92u@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/sdk kind/bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(sdk): use kfp.dsl.types to replace kfp.components.types Fixes #10282 hsinhoyeh/pipelines
1 participant
@hsinhoyeh