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

[docs] batch prediction guide, strict mode #34969

Merged
merged 2 commits into from
May 2, 2023
Merged

[docs] batch prediction guide, strict mode #34969

merged 2 commits into from
May 2, 2023

Conversation

maxpumperla
Copy link
Contributor

No description provided.

@@ -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
Copy link
Contributor

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]".

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed

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.
Copy link
Contributor

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.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Copy link
Contributor

@amogkam amogkam left a 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]>
@amogkam
Copy link
Contributor

amogkam commented May 2, 2023

@ericl your comments are addressed

@amogkam amogkam merged commit 9f35bda into master May 2, 2023
@amogkam amogkam deleted the mp_batch_docs_2 branch May 2, 2023 21:38
architkulkarni pushed a commit to architkulkarni/ray that referenced this pull request May 16, 2023
Signed-off-by: Max Pumperla <[email protected]>
Signed-off-by: amogkam <[email protected]>
Co-authored-by: amogkam <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants