MantisBT - ParaView
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0006725ParaViewBugpublic2008-04-07 09:032016-08-12 09:57
ReporterFilip Sadlo 
Assigned ToBerk Geveci 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionmoved 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Project
Topic Name
Type
Summary0006725: AVSucdReader: wrong node order
DescriptionNode order is converted from AVS to VTK only for the pyramid cell type,
but tets and hexahedra should be converted too (only prism has identical order).

for pyramids the existing code is: vtkAVSucdReader.cxx, line 577:
  //UCD ordering is 0,1,2,3,4 => VTK ordering is 1,2,3,4,0

so for tets it should probably be:
  //UCD ordering is 0,1,2,3 => VTK ordering is 1,2,3,0

and for hexahedra it should probably be:
  //UCD ordering is 0,1,2,3,4,5,6,7 => VTK ordering is 4,5,6,7,0,1,2,3

Both ReadBinaryCellTopology and ReadASCIICellTopology should get corrected.

There are probably other cell types where the node order differs.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2008-04-07 09:03Filip SadloNew Issue
2008-04-11 10:34Filip SadloNote Added: 0011269
2008-04-15 12:52Berk GeveciStatusbacklog => @30@
2008-04-15 12:52Berk GeveciCategory3.4 => 3.6
2009-02-17 14:10Ken MorelandStatus@30@ => tabled
2009-02-17 14:10Ken MorelandAssigned To => Utkarsh Ayachit
2009-02-20 13:14Utkarsh AyachitAssigned ToUtkarsh Ayachit => Berk Geveci
2009-03-19 15:48Berk GeveciNote Added: 0015738
2009-05-13 13:46Utkarsh AyachitTarget Version => 3.6
2009-08-25 18:50Utkarsh AyachitCategory => Bug
2009-08-25 18:50Utkarsh AyachitTarget Version3.6 => 3.6.2
2009-08-31 09:22Utkarsh AyachitTarget Version3.6.2 => 3.8
2016-08-12 09:57Kitware RobotNote Added: 0037561
2016-08-12 09:57Kitware RobotStatusexpired => closed
2016-08-12 09:57Kitware RobotResolutionopen => moved

Notes
(0011269)
Filip Sadlo   
2008-04-11 10:34   
this is a partial fix, add at line 585 of vtkAVSucdReader.cxx:
 else if (ctype[4*i+3] == vtkAVSucdReader::TET)
      { //UCD ordering is 0,1,2,3 => VTK ordering is 1,2,3,0
        *list++ = topology_list[++k2] - 1;
        *list++ = topology_list[++k2] - 1;
        *list++ = topology_list[++k2] - 1;
        *list++ = topology_list[k2-3] - 1;
        k2++;
      }
    else if (ctype[4*i+3] == vtkAVSucdReader::HEX)
      { //UCD ordering is 0,1,2,3,4,5,6,7 => VTK ordering is 4,5,6,7,0,1,2,3
        *list++ = topology_list[k2 + 4] - 1;
        *list++ = topology_list[k2 + 5] - 1;
        *list++ = topology_list[k2 + 6] - 1;
        *list++ = topology_list[k2 + 7] - 1;
        *list++ = topology_list[k2 + 0] - 1;
        *list++ = topology_list[k2 + 1] - 1;
        *list++ = topology_list[k2 + 2] - 1;
        *list++ = topology_list[k2 + 3] - 1;
        k2 += 8;
      }
(0015738)
Berk Geveci   
2009-03-19 15:48   
Do you have any datasets with these cell types? I would like to have a dataset so that I can test the fix.
(0037561)
Kitware Robot   
2016-08-12 09:57   
Resolving issue as `moved`.

This issue tracker is no longer used. Further discussion of this issue may take place in the current ParaView Issues page linked in the banner at the top of this page.