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.

Sign up for GitHub

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

Jump to bottom

PCL cannot load pointclouds written by nerfstudio #1571

Closed
f-dy opened this issue Mar 8, 2023 · 1 comment · Fixed by #1572
Closed

PCL cannot load pointclouds written by nerfstudio #1571

f-dy opened this issue Mar 8, 2023 · 1 comment · Fixed by #1572

Comments

@f-dy
Copy link
Contributor

f-dy commented Mar 8, 2023

We get this when loading nerfstudio's ply with pcl (https://pointclouds.org/):

Failed to find match for field 'x'.
Failed to find match for field 'y'.
Failed to find match for field 'z'.
Failed to find match for field 'normal_x'.
Failed to find match for field 'normal_y'.
Failed to find match for field 'normal_z'.
Failed to find match for field 'curvature'.

The issue is also described here: isl-org/Open3D#1633

it looks like it's just a matter of using the new Open3D Tensor IO o3d.t.io.write_point_cloud instead of the legacy o3d.io.write_point_cloud here

I will test this, but putting this as a reminder.
f-dy added a commit to f-dy/nerfstudio that referenced this issue Mar 9, 2023
@f-dy
scripts/exporter.py: export PLY compatible with PCL
8dcc03c 
Use the newer Open3D Tensor API to write pointclouds as float32, which
are compatible with PCL (see isl-org/Open3D#1633).

fixes nerfstudio-project#1571
f-dy added a commit to f-dy/nerfstudio that referenced this issue Mar 9, 2023
@f-dy
scripts/exporter.py: export PLY compatible with PCL
6de0095 
Use the newer Open3D Tensor API to write pointclouds as float32, which
are compatible with PCL (see isl-org/Open3D#1633).

fixes nerfstudio-project#1571
@f-dy f-dy mentioned this issue Mar 9, 2023
Merged
@f-dy
Copy link
Contributor Author

f-dy commented Mar 9, 2023

PR #1572 resolves this issue

f-dy added a commit to f-dy/nerfstudio that referenced this issue Mar 9, 2023
@f-dy
scripts/exporter.py: export PLY compatible with PCL
f85d6be 
Use the newer Open3D Tensor API to write pointclouds as float32, which
are compatible with PCL (see isl-org/Open3D#1633).

fixes nerfstudio-project#1571
f-dy added a commit to f-dy/nerfstudio that referenced this issue Mar 10, 2023
@f-dy
scripts/exporter.py: export PLY compatible with PCL
6e8b033 
Use the newer Open3D Tensor API to write pointclouds as float32, which
are compatible with PCL (see isl-org/Open3D#1633).

fixes nerfstudio-project#1571
tancik pushed a commit that referenced this issue Mar 10, 2023
@f-dy
scripts/exporter.py: export PLY compatible with PCL (#1572)
21a452f 
* scripts/exporter.py: export PLY compatible with PCL

Use the newer Open3D Tensor API to write pointclouds as float32, which
are compatible with PCL (see isl-org/Open3D#1633).

fixes #1571

* exporter.py: pylint: disable=no-member

fixes https://github.com/nerfstudio-project/nerfstudio/actions/runs/4377569585/jobs/7661246358#step:10:13

open3d.core.Dtype has been around since open3d 0.11.0 http://www.open3d.org/docs/0.11.0/python_api/open3d.core.Dtype.html

---------

Co-authored-by: Frédéric Devernay <f-dy@users.noreply.github.com>
chris838 pushed a commit to chris838/nerfstudio that referenced this issue Apr 22, 2023
@f-dy
scripts/exporter.py: export PLY compatible with PCL (nerfstudio-proje…
ebac7ba 
…ct#1572)

* scripts/exporter.py: export PLY compatible with PCL

Use the newer Open3D Tensor API to write pointclouds as float32, which
are compatible with PCL (see isl-org/Open3D#1633).

fixes nerfstudio-project#1571

* exporter.py: pylint: disable=no-member

fixes https://github.com/nerfstudio-project/nerfstudio/actions/runs/4377569585/jobs/7661246358#step:10:13

open3d.core.Dtype has been around since open3d 0.11.0 http://www.open3d.org/docs/0.11.0/python_api/open3d.core.Dtype.html

---------

Co-authored-by: Frédéric Devernay <f-dy@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

scripts/exporter.py: export PLY compatible with PCL f-dy/nerfstudio
1 participant
@f-dy