Vertex Colors not imported for PLY files in 2.8 #63227

Closed
opened 2019-04-02 22:02:31 +02:00 by David Ortley · 11 comments

System Information
Operating system: Windows-10-10.0.17134 64 Bits
Graphics card: GeForce GTX 1060 6GB/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 418.96

Blender Version
Broken: version: 2.80 (sub 53), branch: blender2.7, commit date: 2019-03-31 21:52, hash: blender/blender@b936d7b16c
Worked: Blender 2.79 and earlier

Short description of error
In Blender 2.79 and prior, importing .ply files which contained vector color information would bring in vertex colors. Importing the same .ply files in Blender 2.8 does not bring in vertex colors with the mesh. See attached image comparing ver2.7 (left) and ver2.8 (right).

Exact steps for others to reproduce the error

I've attached a .ply file that contains vertex color info. To reproduce, 'File->Import->Stanford (.ply)', and select test.ply (also attached.) Vertex colors should exist for the attached .ply file (test.ply)
example.png
test.ply

**System Information** Operating system: Windows-10-10.0.17134 64 Bits Graphics card: GeForce GTX 1060 6GB/PCIe/SSE2 NVIDIA Corporation 4.5.0 NVIDIA 418.96 **Blender Version** Broken: version: 2.80 (sub 53), branch: blender2.7, commit date: 2019-03-31 21:52, hash: `blender/blender@b936d7b16c` Worked: Blender 2.79 and earlier **Short description of error** In Blender 2.79 and prior, importing .ply files which contained vector color information would bring in vertex colors. Importing the same .ply files in Blender 2.8 does not bring in vertex colors with the mesh. See attached image comparing ver2.7 (left) and ver2.8 (right). **Exact steps for others to reproduce the error** I've attached a .ply file that contains vertex color info. To reproduce, 'File->Import->Stanford (.ply)', and select test.ply (also attached.) Vertex colors should exist for the attached .ply file (test.ply) ![example.png](https://archive.blender.org/developer/F6907956/example.png) [test.ply](https://archive.blender.org/developer/F6907958/test.ply)
Author

Added subscriber: @DavidOrtley

Added subscriber: @DavidOrtley

Added subscriber: @intrigus-3

Added subscriber: @intrigus-3

Added subscribers: @JacquesLucke, @rjg

Added subscribers: @JacquesLucke, @rjg

Looks like some code for the vertex color from the 2.79b release was changed in the port to 2.8. I'm not sure why, but there is a comment saying # XXX, colors dont come in right, needs further investigation. in the 2.79b version, so perhaps this feature didn't quite work as expected.

@JacquesLucke do you know why this feature is missing?

Looks like some code for the vertex color from the 2.79b release was changed in the port to 2.8. I'm not sure why, but there is a comment saying `# XXX, colors dont come in right, needs further investigation.` in the 2.79b version, so perhaps this feature didn't quite work as expected. @JacquesLucke do you know why this feature is missing?

The code for handling vertex colors is there, but changes from the 2.79b version were introduced in 227fafdfcf. I'll have to take a closer look.

The code for handling vertex colors is there, but changes from the 2.79b version were introduced in 227fafdfcf4f. I'll have to take a closer look.
Member

@rjg, unfortunately, I don't know why this feature is missing currently.

@rjg, unfortunately, I don't know why this feature is missing currently.

I think I found the bug. The current implementation requires an alpha channel for the vertex color otherwise it's ignored.

I think I found the bug. The current implementation requires an alpha channel for the vertex color otherwise it's ignored.

Added subscriber: @ideasman42

Added subscriber: @ideasman42

The bug was introduced in 1542092295 and @ideasman42 had already noticed it. However there doesn't seem to be any old bug report or patch.

The bug was introduced in 1542092295c9 and @ideasman42 had already noticed it. However there doesn't seem to be any old bug report or patch.
Robert Guetzkow self-assigned this 2019-04-04 22:48:25 +02:00

This issue was referenced by b9ba95de27

This issue was referenced by b9ba95de270f5bffc3b99526a63801f48f3b64bc
Member

Changed status from 'Open' to: 'Resolved'

Changed status from 'Open' to: 'Resolved'
Sign in to join this conversation.
No Milestone
No project
No Assignees
5 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender-addons#63227
No description provided.