-
Notifications
You must be signed in to change notification settings - Fork 39
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
runhouse server dependencies #1652
Conversation
013a72b
to
b74f4ad
Compare
docs/how-to-use-runhouse.rst
Outdated
@@ -12,8 +12,8 @@ Before reviewing this detailed guide, we recommend you start with the `Quick Sta | |||
* Install Runhouse with ``pip install runhouse`` | |||
|
|||
* Optionally install with a specific cloud like ``pip install "runhouse[aws]"`` or with | |||
`SkyPilot <https://skypilot.readthedocs.io/en/latest/docs/index.html>`_ for elastic compute | |||
``pip install "runhouse[sky]"`` | |||
`SkyPilot <https://skypilot.readthedocs.io/en/latest/docs/index.html>`_ for elastic compute with |
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.
after some docs refactoring, we now have
- more comprehensive install + compute setup page
- how to use runhouse -> development guide (directing to installation/quick start for prereqs)
I think we should update this in the installation page. and in the quick-start-local page (standalone, not in the table of contents but is linked out from the installation page) we should update pip install runhouse
to pip install runhouse[server]
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.
code changes look good, made a comment on docs updates
b74f4ad
to
2c26d98
Compare
2c26d98
to
a79690f
Compare
No description provided.