-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
[DataFrame] readthedocs page for Pandas on Ray #1714
Conversation
doc/source/pandas_on_ray.rst
Outdated
@@ -0,0 +1,71 @@ | |||
Pandas on Ray | |||
====== |
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.
These things should be the same length as the title, so
Pandas on Ray
===========
same with the other headings throughout the page
doc/source/pandas_on_ray.rst
Outdated
.. _`installation page`: http://ray.readthedocs.io/en/latest/installation.html | ||
.. _`open an issue`: http://github.com/ray-project/ray/issues | ||
.. _`blog post`: http://rise.cs.berkeley.edu/blog/pandas-on-ray | ||
.. _`GitHub gist`: http://gist.github.com/devin-petersohn/f424d9fb5579a96507c709a36d487f24#file-pandas_on_ray_blog_post_0-ipynb |
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.
this isn't used, right?
doc/source/pandas_on_ray.rst
Outdated
Once you import the library, you should see something similar to the following | ||
output: | ||
|
||
.. code-block:: |
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'd remove this code block, but up to you.
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 have it in there because if they have installed with pip install ray
it won't show this. I only add this because many people keep opening issues on it.
Test PASSed. |
Test PASSed. |
Test PASSed. |
Merging because linting passed. |
No description provided.