View Issue Details [ Jump to Notes ] [ Print ]
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 
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.
TagsNo tags attached.
Project
Topic Name
Type
Attached Files

 Relationships

  Notes
(0011269)
Filip Sadlo (reporter)
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 (administrator)
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 (administrator)
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.

 Issue History
Date Modified Username Field Change
2008-04-07 09:03 Filip Sadlo New Issue
2008-04-11 10:34 Filip Sadlo Note Added: 0011269
2008-04-15 12:52 Berk Geveci Status backlog => @30@
2008-04-15 12:52 Berk Geveci Category 3.4 => 3.6
2009-02-17 14:10 Ken Moreland Status @30@ => tabled
2009-02-17 14:10 Ken Moreland Assigned To => Utkarsh Ayachit
2009-02-20 13:14 Utkarsh Ayachit Assigned To Utkarsh Ayachit => Berk Geveci
2009-03-19 15:48 Berk Geveci Note Added: 0015738
2009-05-13 13:46 Utkarsh Ayachit Target Version => 3.6
2009-08-25 18:50 Utkarsh Ayachit Category => Bug
2009-08-25 18:50 Utkarsh Ayachit Target Version 3.6 => 3.6.2
2009-08-31 09:22 Utkarsh Ayachit Target Version 3.6.2 => 3.8
2016-08-12 09:57 Kitware Robot Note Added: 0037561
2016-08-12 09:57 Kitware Robot Status expired => closed
2016-08-12 09:57 Kitware Robot Resolution open => moved

Copyright © 2000 - 2018 MantisBT Team