Skip to content

Fix remove to legacy in docs#7431

Open
h812755591 wants to merge 2 commits intoisl-org:mainfrom
h812755591:fix-remove-to-legacy-in-docs
Open

Fix remove to legacy in docs#7431
h812755591 wants to merge 2 commits intoisl-org:mainfrom
h812755591:fix-remove-to-legacy-in-docs

Conversation

@h812755591
Copy link

just one file docs/jupyter/t_geometry/pointcloud.ipynb
1,
remove pointcloud.to_legacy method
2,
Conversion between tensor and legacy point cloud ===>Conversion between tensor and point cloud
PointCloud can be converted to/from legacy open3d.geometry.PointCloud.====>PointCloud can be converted to Tensor

@update-docs
Copy link

update-docs bot commented Feb 8, 2026

Thanks for submitting this pull request! The maintainers of this repository would appreciate if you could update the CHANGELOG.md based on your changes.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@h812755591
Copy link
Author

This fixes outdated documentation: to_legacy() is no longer part of the Tensor PointCloud API.
The updated wording reflects the current conversion behavior

@h812755591
Copy link
Author

This is a documentation-only change with no user-facing behavior change,
so I did not update the CHANGELOG.

Copy link
Member

@ssheorey ssheorey left a comment

Choose a reason for hiding this comment

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

Hi @h812755591 thanks for submitting this PR. However, we don't have support for tensor point clouds in draw_geometries(). You have to use the new draw() function, which supports both legacy and tensor point clouds.

@ssheorey ssheorey added the status / needs info Waiting for information from reporter / author label Feb 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status / needs info Waiting for information from reporter / author

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants