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

Parallelizing ply parser #26

Open
goldragoon opened this issue Jul 29, 2020 · 1 comment
Open

Parallelizing ply parser #26

goldragoon opened this issue Jul 29, 2020 · 1 comment

Comments

@goldragoon
Copy link

Hi, I'm recently working with analysis of large PLY-formatted mesh model. Since, 3D model I/O is really big bottleneck of my analysis pipeline, I'm trying to parallelizing ply parser based on this repository using the idea of [1].
If you think that this feature is worth and suitable to this repository, I'd like to contribute my implementation here.

Thanks for reading this issue.

Reference
[1] Jo, Sunghun, Yuna Jeong, and Sungkil Lee. "GPU-driven scalable parser for OBJ models." Journal of Computer Science and Technology 33.2 (2018): 417-428.

@nmwsharp
Copy link
Owner

Hi!

I'd definitely be interested in seeing any efforts to parallelize happly, though before merging such code in to the parser I'd have two concerns:

  • There are other parts of the happly code which are probably lower-hanging fruit for performance improvements. In particular, ASCII float parsing and irregular memory accesses could both be dramatically improved. One would probably want to tackle those simultaneously to create an actual "high performance" happly.
  • We'd need to "guard" it somehow, to the parallelization doesn't add any software burden to users who don't want it. This could probably be done with #ifdef.

For what it's worth, happly is neither the simplest, nor the fastest PLY parsing library out there; instead, happly is focused on being simple, and handling general non-mesh data. If you're interested in high-performance PLY parsing, you might want to look elsewhere, @mhalber has a great benchmark here: https://github.com/mhalber/ply_io_benchmark (though happly has gotten a bit faster since those timings).

Best,
Nick

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

No branches or pull requests

2 participants