MantisBT - ParaView
View Issue Details
0015997ParaView(No Category)public2016-02-12 20:412016-06-09 10:58
Alan Scott 
Utkarsh Ayachit 
normalminorhave not tried
closedfixed 
4.4 
5.15.1 
Sandia
incorrect functionality
0015997: PLY writer and/or reader is broken.
The PLY writer and/or reader is broken. Here is how to replicate:

* 4.4.0, local server, Linux.
* Load z_pinch1.8.[0-7]. All vars on. Apply. (Ask Utkarsh for this file set.)
* Go to last timestep. Paint by J.
* Filters/ Extract Surface. Apply.
* File / Save Data/ ply. Write this file somewhere.
* Edit/ Reset Session.
* Load Ply file. It won't have correct colors - everything will be white. Be sure to try to color by a variable.
No tags attached.
Issue History
2016-02-12 20:41Alan ScottNew Issue
2016-02-18 09:11Cory QuammenNote Added: 0035751
2016-02-19 13:33Alan ScottNote Added: 0035755
2016-02-19 13:34Alan ScottNote Added: 0035756
2016-02-22 17:44Alan ScottNote Added: 0035761
2016-02-22 17:44Alan ScottTarget Version => 5.1
2016-03-01 12:50Alan ScottNote Added: 0035804
2016-03-30 15:45Utkarsh AyachitAssigned To => Utkarsh Ayachit
2016-03-30 15:45Utkarsh AyachitStatusbacklog => active development
2016-03-31 16:16Utkarsh AyachitNote Added: 0035868
2016-03-31 16:16Utkarsh AyachitStatusactive development => gatekeeper review
2016-03-31 16:16Utkarsh AyachitResolutionopen => fixed
2016-03-31 16:26Utkarsh AyachitNote Added: 0035869
2016-03-31 16:27Utkarsh AyachitNote Edited: 0035869bug_revision_view_page.php?bugnote_id=35869#r1559
2016-03-31 21:15Utkarsh AyachitStatusgatekeeper review => customer review
2016-03-31 21:15Utkarsh AyachitFixed in Version => git-master
2016-04-15 22:17Alan ScottNote Added: 0035892
2016-04-15 22:17Alan ScottStatuscustomer review => closed
2016-06-09 10:58Utkarsh AyachitFixed in Versiongit-master => 5.1

Notes
(0035751)
Cory Quammen   
2016-02-18 09:11   
Alan,

The PLY format is not general enough to save point or cell data. It does support point colors, and it should be possible to export colors mapped through the current lookup table with some work.

However, when you load the PLY, it will not have the original variables by which you could color. It will be limited to either solid color or the colors on the surface as they appeared when the file was saved.

Would you like to have the functionality of saving the colors in the PLY file?
(0035755)
Alan Scott   
2016-02-19 13:33   
Yes, I would like the functionality you describe - adding the ability to save the surface color of the object - into PLY files. You are correct - this file format does not include variable arrays, only surface color at the time the file was saved.
(0035756)
Alan Scott   
2016-02-19 13:34   
I think I found the PLY file format spec. It is here: http://paulbourke.net/dataformats/ply/ [^]
(0035761)
Alan Scott   
2016-02-22 17:44   
Needed to create products to be ingested by slycat.
(0035804)
Alan Scott   
2016-03-01 12:50   
I have another user trying to use this. Processing the output of the new CTH contour functionality.
(0035868)
Utkarsh Ayachit   
2016-03-31 16:16   
MR: https://gitlab.kitware.com/paraview/paraview/merge_requests/713 [^]


    BUG 0015997: Add ability to save color information in PLY files.
    
    The PLY writer now allows user to choose whether color information is to
    be saved in the output file. When enabled, ParaView uses the data
    sources coloring state in the active view i.e. the array to color with
    and the LUT to use, to pass along to the writer so that it can save RGB
    color values in the output dataset.
(0035869)
Utkarsh Ayachit   
2016-03-31 16:26   
(edited on: 2016-03-31 16:27)
Note that on reloading the PLY file in ParaView, ParaView will by default use a color map for coloring and hence the color will look different. Uncheck "Map Scalars" and you'll see the RGB colors saved in the PLY file which should match the color map used when saving the PLY file.

(0035892)
Alan Scott   
2016-04-15 22:17   
Yes, I believe that is what the user wanted. Nice, thanks.

Tested local server, master, Linux.