-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Ply dataparser #2557
Ply dataparser #2557
Conversation
Nice, this is useful. I think we can merge the dataparser with its own PR, and then a separate PR should add code to ns-process-data to generate a PLY from colmap when it finishes. |
I think this is useful enough to merge into main, since it includes generally loading .ply files in the dataparser and also metric export option |
Need to pull out just the relevant changes though, changing the base branch includes all of splatting too |
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.
sgtm
IN PROGRESS do not merge yet
Loads a ply file as initial point cloud if specified
Specify
transforms.json
with "ply_file_path" in header, i.e{
"camera_model": "OPENCV",
"cx": 320,
"cy": 240,
"w": 640,
"h": 480,
"ply_file_path": "smith.ply",
...