MantisBT - VTK
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0000405VTK(No Category)public2003-11-25 09:362016-08-12 09:54
Reporterchuck yagger 
Assigned ToBerk Geveci 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionmoved 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Project
Type
Summary0000405: vtkPointSet::FindCell
DescriptionHello,

I use vtkStreamTrace to computes stream lines on many sources (at least 2) using AddSource.

Sometimes streamlines stopped at the interface.

I have change this part of code in vtkPointSet::FindCell an i have better results.
-------------------------
if ( cellIds->GetNumberOfIds() > 0 )
      {
      cellId = cellIds->GetId(0); //arbitrarily use first cell in list
-------------------------
by
-------------------------
int i;
for ( i = 0 ; i < cellIds->GetNumberOfIds() ; i++ )
{cellId = cellIds->GetId(i);
-------------------------


CIAOOOO
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Filespatch pointset.patch (710) 1969-12-31 19:00
https://www.vtk.org/Bug/file/5028/pointset.patch
gz BUG_VTK_STR.tar.gz (286,317) 1969-12-31 19:00
https://www.vtk.org/Bug/file/5036/BUG_VTK_STR.tar.gz
txt CMakeLists.txt (178) 1969-12-31 19:00
https://www.vtk.org/Bug/file/5047/CMakeLists.txt
Issue History
Date ModifiedUsernameFieldChange
2008-03-06 18:59Berk GeveciNote Deleted: 0002360
2011-06-16 13:11Zack GalbreathCategory => (No Category)
2016-08-12 09:54Kitware RobotNote Added: 0036702
2016-08-12 09:54Kitware RobotStatusexpired => closed
2016-08-12 09:54Kitware RobotResolutionopen => moved

Notes
(0000542)
Mathieu Malaterre   
2004-01-21 18:34   
Berk, could you comment on that patch. I create an attachment 'more formal'. Thanks.
(0000551)
chuck yagger   
2004-01-22 03:17   
test
(0000552)
chuck yagger   
2004-01-22 03:23   
Berk too.

In french:
Ce patch permet lors du calcul de ligne de courant de permettre le passage entre deux sources differentes. La version precedante utilisait "arbitrairement" la premiere cellule. La version corrigé balaye toutes les cellules.

In "poor" english:
This patch is usefull when computing "stream" on many sources. It allow to change of source by checking whole "cells" instead of only the first.

bye. ;-D
(0000558)
Mathieu Malaterre   
2004-01-22 13:39   
Could you please send us a demo dataset. That would allow us:
1. To reproduce the bug
2. Check if it worse it in term of speed, because there is a huge overhead if we apply this patch.
(0000560)
chuck yagger   
2004-01-23 03:28   
I realize that a test case would be usefull for you. My test case is an industrial "unshowable" dataset. I'm going to try to build another test case.
I suggest you to activate this patch optionnaly (if it's possible), while we can't proove that it's necessary.
But the comment in the original file (//arbitrarily use first cell in list ) let me think that I'm right.

Thank you for your work. I will do all what i could.
(0000561)
Mathieu Malaterre   
2004-01-23 10:42   
From Berk:
We have been aware of issues with FindCell. However, the patch you suggest is very inefficient and can't be used. We will (eventually) fix this issue. I know of a few cases when FindCell fails. However, it would be very useful to have a test case that shows the problem you ran into since it doesn't sound like a case I ran into before.
(0000563)
chuck yagger   
2004-01-23 12:09   
The test case is under construction.
I will send it next week.

Have a nice week end.
(0000595)
chuck yagger   
2004-02-02 09:39   
Hello,

BUG_VTK_STR.tar.gz is the test case with source code, makefile (for sun) and readme file.

I hope it would be usefull.

Thank again for all yours work.

I was wrong in one mail (I use vtkStreamTracer).

Bye.
(0000661)
Mathieu Malaterre   
2004-02-17 17:35   
Berk, could you comment on that ? Just extract BUG_VTK_STR.tar.gz and use the CMakeLists.txt I have added.
(0000727)
chuck yagger   
2004-03-04 04:02   
Does any body have tested BUG_VTK_STR.tar.gz ??
Or I have maid it for nothing ???
I'm still interesting by yours comments.

Thank a lot.
(0036702)
Kitware Robot   
2016-08-12 09:54   
Resolving issue as `moved`.

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