(0000542)
Mathieu Malaterre (developer)
2004-01-21 18:34

Berk, could you comment on that patch. I create an attachment 'more formal'. Thanks.

(0000551)
chuck yagger (reporter)
2004-01-22 03:17

test

(0000552)
chuck yagger (reporter)
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 (developer)
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 (reporter)
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 (developer)
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 (reporter)
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 (reporter)
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 (developer)
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 (reporter)
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 (administrator)
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.

Issue History
Date Modified	Username	Field	Change
2008-03-06 18:59	Berk Geveci	Note Deleted: 0002360
2011-06-16 13:11	Zack Galbreath	Category	=> (No Category)
2016-08-12 09:54	Kitware Robot	Note Added: 0036702
2016-08-12 09:54	Kitware Robot	Status	expired => closed
2016-08-12 09:54	Kitware Robot	Resolution	open => moved

Notes
(0000542) Mathieu Malaterre (developer) 2004-01-21 18:34	Berk, could you comment on that patch. I create an attachment 'more formal'. Thanks.

(0000551) chuck yagger (reporter) 2004-01-22 03:17	test

(0000552) chuck yagger (reporter) 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 (developer) 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 (reporter) 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 (developer) 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 (reporter) 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 (reporter) 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 (developer) 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 (reporter) 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 (administrator) 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.

Relationships