-
Notifications
You must be signed in to change notification settings - Fork 6k
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] batch prediction guide, strict mode #34969
Conversation
Signed-off-by: Max Pumperla <[email protected]>
@@ -498,6 +497,18 @@ namely Pandas, NumPy, and Arrow, and how they're used in Ray Data: | |||
:start-after: __simple_numpy_start__ | |||
:end-before: __simple_numpy_end__ | |||
|
|||
.. tabbed:: Pandas |
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.
Can we remove the entire paragraph above (simple vs tensor vs tabular)? We can replace it saying "NumPy batch format always presents batches as Dict[str, np.ndarray]".
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.
removed
doc/source/data/batch_inference.rst
Outdated
The ``"pandas"`` batch format presents batches in | ||
`pandas.DataFrame <https://pandas.pydata.org/docs/reference/api/pandas.DataFrame.html>`__ | ||
format. If converting a simple dataset to Pandas DataFrame batches, a single-column | ||
dataframe with the column ``"__value__"`` will be created. |
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.
Remove all mention of __value__
or "single-column" datasets.
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
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.
awesome, thanks a lot! lgtm once eric's comments are addressed
Signed-off-by: amogkam <[email protected]>
@ericl your comments are addressed |
Signed-off-by: Max Pumperla <[email protected]> Signed-off-by: amogkam <[email protected]> Co-authored-by: amogkam <[email protected]>
No description provided.