Skip to content

Add section on workflows and mlflow-builder workflow extension - #49

Merged
stefannica merged 1 commit into
fuseml:mainfrom
stefannica:mlflow-builder
Dec 2, 2021
Merged

stefannica merged 1 commit into
fuseml:mainfrom
stefannica:mlflow-builder

Conversation

@stefannica

Copy link
Copy Markdown
Member

Comment thread docs/workflows/workflows.md Outdated

This is where the workflow's global parameters, inputs and outputs are declared. Currently, the only type of supported input for FuseML workflows and steps is the codeset. In this example:

- the `mlflow-codeset` input is a codeset that must contain an MLflow project with python code that will be used to train a ML model

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It might be a good idea to add a link to the description of codesets

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

done (I hope)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hm, I do not see any reference there. I had in mind a reference to codeset definition, but maybe we do not have one...

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

no, we don't have a section dedicated to codesets :-(

jsuchome
jsuchome previously approved these changes Dec 1, 2021

@jsuchome jsuchome left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice

@stefannica
stefannica force-pushed the mlflow-builder branch 4 times, most recently from c32fabc to b05fef3 Compare December 1, 2021 16:22
Comment thread docs/workflows/kserve-predictor.md
Comment thread docs/workflows/kserve-predictor.md
Comment thread docs/workflows/mlflow-builder.md Outdated
@stefannica stefannica added the needs-backport PR needs to be back-ported to a previous release label Dec 2, 2021
@stefannica
stefannica merged commit ba3071c into fuseml:main Dec 2, 2021
@stefannica
stefannica deleted the mlflow-builder branch December 2, 2021 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-backport PR needs to be back-ported to a previous release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants