-
Notifications
You must be signed in to change notification settings - Fork 196
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 norm consideration #27
Comments
I have never seen a point cloud that has normals. How did you create it? |
Meshlab produced it for me, and i don't want to ignore / disregard them. |
Could you provide a .ply point cloud data file with normals? |
Sure, Because it was created from a PLY without normals originally, the normals don't add information, but taking a PLY with normal information and disregarding it affects the lighting from different camera angles and different lightings. |
I realized that it needs some changes in the basic design of the importer. It requires some amount of time to support it. I'd like to add this to the to-do list of the project. Unfortunately, I can't afford time for it at the moment. I'll revisit it when I get free time. |
Sure, I get it. |
Add https://github.com/keijiro/Pcx/blob/master/Assets/Pcx/Runtime/Shaders/Point.shader#L30 Then you can use the normal in the vertex shader. |
Thanks |
I have PLY with normals as well. Perhaps I could help get this working? |
I would also love to use ply normals in the VFX graph, in the same way you would with a pcache file |
It would be great to use normals from the .ply file to set the particle direction in VFX graph. Would this require generating a normals texture, in addition to colour and position maps? |
I noticed that while parsing the PLY meta data, the norm is not taken into consideration explicitly (only through a generic Data32 type).
Later, after parsing the data, only the vertices and colors (rgba) are moved on to the shader.
I have a PLY with norm i need to take into consideration.
How do i implement using it in the shader? (point/disk)
Thanks
The text was updated successfully, but these errors were encountered: