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

Avoid unnecessary data copy in ply reader #3

Closed
aecins opened this issue May 8, 2017 · 1 comment
Closed

Avoid unnecessary data copy in ply reader #3

aecins opened this issue May 8, 2017 · 1 comment
Labels
enhancement

Comments

@aecins
Copy link
Collaborator

aecins commented May 8, 2017

Current implementation of readPointCloudFromPLYFile first reads pointcloud data to local variables, and then copies this data to to a PointCloud data structure. A better implementation would avoid this unnecessary data copy and would write directly to the PointCloud.
@kzampog
Copy link
Owner

kzampog commented Oct 4, 2017

It seems there is no (straightforward) way of doing this without modifying tinyply itself :(

@kzampog kzampog closed this as completed Oct 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@aecins @kzampog