-
Notifications
You must be signed in to change notification settings - Fork 239
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
[Docs] Make mlrun-kit the main installation option #590
Conversation
d048052
to
9be65cd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
several minor comments
README.md
Outdated
@@ -71,7 +91,7 @@ pip install mlrun | |||
MLRun requires separate containers for the API and the dashboard (UI). | |||
You can also select to use the pre-baked JupyterLab image. | |||
|
|||
To install and run MLRun locally using Docker or Kubernetes, see the instructions in [**hack/local/README.md**](hack/local/README.md). | |||
To install and run MLRun locally using Docker or Kubernetes, see the instructions in the [**MLRun documentation**]([hack/local/README.md](https://mlrun.readthedocs.io/en/latest/install.html)). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The new link doesn't work, looks like a typo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
README.md
Outdated
- [Using the MLRun CLI](#run-mlrun-service-cli) | ||
- [Benefits](#benefits) | ||
- [Components](#components) | ||
- [In This Document](#in-this-document) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In this document seems unrelated as a sub-title of components suggest to return it to the top as before plus changing it from ####
to ##
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
docs/install.md
Outdated
|
||
Create a namespace for mlrun. For example: | ||
To install the chart with the release name `my-mlrun` use the following command. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would have rename my-mlrun
to mlrun-kit
no functional change just feels more professional IMO
this is just a suggestion, you don't have to take it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
docs/install.md
Outdated
helm install -n mlrun nfsprov stable/nfs-server-provisioner | ||
#### Set Environment Variables | ||
|
||
Define your MLRun configuration. As a minimum requirement — |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the —
is on purpose ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed (also in remote.md)
No description provided.