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

Revert "[Datasets] Fix ndarray representation of single-element ragged tensor slices." #30709

Conversation

stephanie-wang
Copy link
Contributor

Reverts #30514

This is causing linux://python/ray/data:tests/test_transform_pyarrow to fail.

@stephanie-wang
Copy link
Contributor Author

cc @clarkzinzow.

Copy link
Contributor

@architkulkarni architkulkarni left a comment

Choose a reason for hiding this comment

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

Stamp

@clarkzinzow
Copy link
Contributor

@stephanie-wang Btw got a tracking issue for this: #30690

@stephanie-wang stephanie-wang merged commit 579770a into master Nov 28, 2022
@stephanie-wang stephanie-wang deleted the revert-30514-datasets/fix/ragged-tensor-single-element-slice branch November 28, 2022 22:32
clarkzinzow added a commit to clarkzinzow/ray that referenced this pull request Nov 29, 2022
WeichenXu123 pushed a commit to WeichenXu123/ray that referenced this pull request Dec 19, 2022
…d tensor slices. (ray-project#30514)" (ray-project#30709)

This reverts commit 36aebcb.

Reverts ray-project#30514

This is causing linux://python/ray/data:tests/test_transform_pyarrow to fail.

Signed-off-by: Weichen Xu <weichen.xu@databricks.com>
tamohannes pushed a commit to ju2ez/ray that referenced this pull request Jan 25, 2023
…d tensor slices. (ray-project#30514)" (ray-project#30709)

This reverts commit 36aebcb.

Reverts ray-project#30514

This is causing linux://python/ray/data:tests/test_transform_pyarrow to fail.

Signed-off-by: tmynn <hovhannes.tamoyan@gmail.com>
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.

3 participants