View Issue Details [ Jump to Notes ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0014308ParaView(No Category)public2013-09-26 10:402015-05-27 08:02
ReporterAlejandro Ribes 
Assigned ToJoachim Pouderoux 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version4.1 
Target VersionFixed in Version4.3 
Summary0014308: Invalid surface and "Surface With Edges" rendering of nonconvex VTK_POLYGON
Description0- Open merge.vtu file in attachment
1- Activate Surface or Surface With Edges renderning
-> The rendering of nonconvex polygon cell #0 is invalid (see black part
between convex hull of cell # 0 and real cell in
SurfaceRenderOfNonConvexPolygon.png)

However on page 127 of 4th edition of VTK book in Cell Types section, it
is written that : "The polygon may be nonconvex,...".

Regards,

Alejandro.
TagsNo tags attached.
ProjectParaViS
Topic Name14308_fix_nonconvex_polygon_vis
Typeincorrect functionality
Attached Files? file icon merge.vtu [^] (1,289 bytes) 2013-09-26 10:40
png file icon SurfaceRenderOfNonConvexPolygon.png [^] (3,778 bytes) 2013-09-26 10:42


? file icon merge-polydata.vtk [^] (552 bytes) 2014-09-12 03:50
? file icon merge-unstructured.vtu [^] (1,289 bytes) 2014-09-12 03:50

 Relationships

  Notes
(0031677)
Utkarsh Ayachit (administrator)
2013-10-03 15:21

 Alejandro, while non-convex polygonal cells are supported, they cannot be directly rendered currently. One has to triangulate the surface (by applying "ExtractSurface" and then "Traingulate" filter).

We could potentially add support to the geometry filter in ParaView, to optionally triangulate the cells if requested and use logic similar to what is gone for tessellating higher order elements to avoid showing articial seams for the internal cells resulting from the triangulation. What do you think?
(0031686)
Alejandro Ribes (manager)
2013-10-09 09:00

 Hi Utkarsh,

I contacted some colleagues of the CEA because they found this bug and I copy here their email with the answer.

Alejandro.

-------------------- EMAIL:
Bonjour Alejandro,

I'll do it in English in case that might help for copy/paste :-)

Surely we lack some precise knowledge on the Rendering parts in
Paraview, but indeed, with Anthony we were more seeing this as a new
Rendering mode. We can add it as a plugin. We thus leave the current
Rendering untouched, and we add an extra one using almost the same chain:

- vtkGeometryFilter: to convert everything into polygons
- a new filter doing the triangulation (actually it is already there?)
*and* keeping the mapping between the triangles and the initial polygon
being split
- the final Mapper doing the transformation from the VTK polygonal
structure onto the OpenGL structures

A key point is the selection: not too sure how this works precisely, but
for the selection to work properly, a mapping between what the user
selects (a OpenGL triangle on the screen) and the initial cell (a
concave polygon) has to be maintained.

Having the whole thing as a new plugin will ease potential future
developments for quadratic elements for example.

Let us know if you agree,
Adrien.
(0031704)
Utkarsh Ayachit (administrator)
2013-10-21 07:53

 Alejandro,

Should I have someone work on this task then? I'd vote for incorporating the code in ParaView proper (rather than relying on a plugin).

Utkarsh
(0033378)
Joachim Pouderoux (developer)
2014-09-26 05:44
edited on: 2014-09-26 05:44

 Change Ib9ab09f8: Enhance geometry representation to support non convex polygon rendering.

A new "Triangulate" option is added to the representation.
The work is then done in the geometry filter. If the option is enabled
a vtkTriangleFilter is applied to the mesh (for unstructured & polydata
inputs). The support for picking and correct edges displayed is performed
using the same technique as with non linear cells - original cell ids
are used to compute boundary edges which are then associate to the
mesh point data as an EDGEFLAG field, used downstream by the mapper.

Tested in built-in, client, server and parallel server modes.

Change-Id: Ib9ab09f84b78a827ed94cbd92f667b49ac3beebb
(0033382)
Utkarsh Ayachit (administrator)
2014-09-30 08:28

 Topics merged into master:
        14308_fix_nonconvex_polygon_vis
(VTK) bar-chart-colors
(0033777)
Alan Scott (manager)
2014-11-06 20:30

 Using datasets provided in this bug, it was tested Linux, master, local server.

 Issue History
Date Modified Username Field Change
2013-09-26 10:40 Alejandro Ribes New Issue
2013-09-26 10:40 Alejandro Ribes Assigned To => Utkarsh Ayachit
2013-09-26 10:40 Alejandro Ribes File Added: merge.vtu
2013-09-26 10:42 Alejandro Ribes File Added: SurfaceRenderOfNonConvexPolygon.png
2013-10-03 15:21 Utkarsh Ayachit Note Added: 0031677
2013-10-09 09:00 Alejandro Ribes Note Added: 0031686
2013-10-21 07:53 Utkarsh Ayachit Note Added: 0031704
2014-09-11 11:44 Joachim Pouderoux Assigned To Utkarsh Ayachit => Joachim Pouderoux
2014-09-11 11:44 Joachim Pouderoux Status backlog => active development
2014-09-12 03:49 Joachim Pouderoux Reproducibility have not tried => always
2014-09-12 03:50 Joachim Pouderoux File Added: merge-polydata.vtk
2014-09-12 03:50 Joachim Pouderoux File Added: merge-unstructured.vtu
2014-09-25 12:10 Joachim Pouderoux Topic Name => 14308_fix_nonconvex_polygon_vis
2014-09-26 05:44 Joachim Pouderoux Note Added: 0033378
2014-09-26 05:44 Joachim Pouderoux Note Edited: 0033378
2014-09-26 10:20 Joachim Pouderoux Status active development => gatekeeper review
2014-09-30 08:28 Utkarsh Ayachit Note Added: 0033382
2014-09-30 08:28 Utkarsh Ayachit Status gatekeeper review => customer review
2014-09-30 08:28 Utkarsh Ayachit Fixed in Version => git-master
2014-10-02 06:42 Joachim Pouderoux Resolution open => fixed
2014-11-06 20:30 Alan Scott Note Added: 0033777
2014-11-06 20:30 Alan Scott Status customer review => closed
2015-01-02 10:45 Utkarsh Ayachit Fixed in Version git-master => 4.3
2015-05-07 10:30 Joachim Pouderoux Status closed => backlog
2015-05-07 10:30 Joachim Pouderoux Resolution fixed => reopened
2015-05-07 11:57 Jake Stookey File Added: test.txt
2015-05-07 11:57 Jake Stookey File Deleted: test.txt
2015-05-07 11:59 Matthew Bowman File Added: test.txt
2015-05-07 11:59 Matthew Bowman File Deleted: test.txt
2015-05-07 12:03 Jake Stookey File Added: test.txt
2015-05-07 12:03 Jake Stookey File Deleted: test.txt
2015-05-07 13:51 Jake Stookey File Added: test.txt
2015-05-07 13:51 Jake Stookey File Deleted: test.txt
2015-05-27 04:12 Joachim Pouderoux Status backlog => gatekeeper review
2015-05-27 04:12 Joachim Pouderoux Resolution reopened => fixed
2015-05-27 08:02 Utkarsh Ayachit Status gatekeeper review => customer review
2015-05-27 08:02 Utkarsh Ayachit Status customer review => closed

Copyright © 2000 - 2018 MantisBT Team