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

Add python interpreter into vscode settings #3014

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

thomasjpfan
Copy link
Member

@thomasjpfan thomasjpfan commented Dec 19, 2024

Why are the changes needed?

This PR configures VSCode to use the Python that launched flyte.

What changes were proposed in this pull request?

This PR adds a settings.json file to the vscode configuration.

How was this patch tested?

I ran pyflyte run --remote main.py wf

from flytekit import ImageSpec, task, workflow

new_flytekit = "git+https://github.com/thomasjpfan/flytekit@6a9506bd6fc6ee0590211c720f4e4a64ddbd2e8f"
image_spec = ImageSpec(registry="ghcr.io/thomasjpfan", packages=[new_flytekit], apt_packages=["git"])


@task(container_image=image_spec)
def train():
    print("forward")
    print("backwardd")
    raise RuntimeError("fail")


@workflow
def wf():
    train()

Screenshots

When launching debug I get:

SCR-20241219-kzqt

Copy link
Collaborator

@EngHabu EngHabu left a comment

Choose a reason for hiding this comment

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

Thank you!

@pingsutw pingsutw merged commit e9a7da1 into flyteorg:master Dec 19, 2024
102 checks passed
shuyingliang pushed a commit to shuyingliang/flytekit that referenced this pull request Dec 20, 2024
eapolinario pushed a commit that referenced this pull request Dec 23, 2024
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.

4 participants