Skip to content

Conversation

fiedlerNr9
Copy link
Contributor

@fiedlerNr9 fiedlerNr9 commented Jun 5, 2025

Add kubeflow migration guide.

Copy link

cloudflare-workers-and-pages bot commented Jun 5, 2025

Deploying docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: f5a541a
Status: ✅  Deploy successful!
Preview URL: https://4091b8fc.docs-dog.pages.dev
Branch Preview URL: https://jan-add-kubeflow-migration-g.docs-dog.pages.dev

View logs

@@ -0,0 +1,112 @@
from flytekit import workflow, task, FlyteDirectory, ImageSpec, FlyteFile, Resources
Copy link
Contributor

Choose a reason for hiding this comment

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

The best place to put runnable code is in unionai-examples repo.

- By leveraging **FlyteDirectories**, we reduced the number of tasks from two to one — simplifying the overall structure.
- …and much more!

{{< download "/docs/flyte/code/example_flyte_workflow.py" "example_flyte_workflow.py" >}}
Copy link
Contributor

@nelsonjr nelsonjr Jun 5, 2025

Choose a reason for hiding this comment

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

Do not hardcode absolute paths or move them to /content/_static/ and refer as /_static/... like here: https://github.com/unionai/docs/blob/d04fde5fda8a6d24dba1878aea839cffc261e819/content/deployment/single-sign-on-setup/google-oidc.md?plain=1#L43

Instead, I suggest making the examples actual tutorial examples, like this one: https://github.com/unionai/docs/blob/main/content/tutorials/finance/credit-default-xgboost.md?plain=1.

You document the code and add them as markdown in this repository.

Copy link
Contributor

Choose a reason for hiding this comment

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

@ppiegaze FYI and for guidance for Jan.

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