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

MLflow / Kubeflow Tutorial Conflation #202

Open
ColmBhandal opened this issue Sep 14, 2023 · 0 comments
Open

MLflow / Kubeflow Tutorial Conflation #202

ColmBhandal opened this issue Sep 14, 2023 · 0 comments
Labels
documentation Improvements or additions to documentation

Comments

@ColmBhandal
Copy link
Contributor

The MLflow tutorial mlflow.rst is today written in a way that also supports the mlflow-kubeflow.rst tutorial. This introduces some issues:

Issue 1

In mlflow-kubeflow.rst, we have this:

Follow the steps in this tutorial to deploy MLflow on your VM: :doc:mlflow

This is not a good UX - sending them off to a separate doc.

Issue 2

In mlflow.rst, we have this:

We're just using kubeflow because often you may want to deploy MLflow along with Kubeflow, and in that case, the model name must be kubeflow. So it's not a bad habit to have.

There are also other references to Kubeflow. This runs the risk of confusing the reader by conflating the pure MLflow tutorial with Kubeflow.

@ColmBhandal ColmBhandal added the documentation Improvements or additions to documentation label Sep 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
Status: Labeled
Development

No branches or pull requests

1 participant