View Issue Details [ Jump to Notes ]

[ Print ]

Project

Category

View Status

Date Submitted

Last Update

0000962

VTK

(No Category)

public

2004-06-28 12:47

2016-08-12 09:54

Reporter

Mathieu Malaterre

Assigned To

Kitware Robot

Priority

low

Severity

feature

Reproducibility

always

Status

closed

Resolution

moved

Platform

OS Version

Product Version

Target Version

Fixed in Version

Summary

0000962: Bug in vtkPicker

Description

Hi,

The pick function in VTK is some unstable. I tried to do some programming to
pick a vtkProp in a vtkAssembly.
it turned out to crash after i picked several times. I tried to track the
code in VTK, and found vtkPicker has some suspicious codes.
in version(4.2.1) line 300 of vtkPicker.cxx:
      this->Prop3Ds->AddItem((vtkProp3D *)prop);
I guess something is wrong there. it will collect the assembly many times.
now I found it was updated with
the codes in CVS :
                                  if ( ! this->Prop3Ds->IsItemPresent(prop)
)
368 {
369 this->Prop3Ds->AddItem((vtkProp3D *)prop);
370 }

I think it is better to change as:
                                   if ( !
this->Prop3Ds->IsItemPresent(propCandidate) )
368 {
369 this->Prop3Ds->AddItem((vtkProp3D
*)propCandidate);
370 }

As i read the code, I think prop is a vtkAssembly and propCandidate is a
vtkProp composited in the assembly.
before we add this item to Props3Ds, propCandidate's bounding box is checked
for the intersection test. So i think it
is rational to add propCandidate to Props3Ds.

Do we have some serious picking test with vtkAssembly involved?

I'm not sure my opinion is correct or wrong. anyway, i hope vtk has a robust
picking function.
any hints will be helpful.

Thanks

Dennis

Tags

No tags attached.

Project

Type

Attached Files

Relationships

Notes
(0036738) 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-11-30 09:26	Mathieu Malaterre	Assigned To	Mathieu Malaterre => François Bertel
2008-11-30 20:24	François Bertel	Assigned To	François Bertel =>
2011-06-16 13:11	Zack Galbreath	Category	=> (No Category)
2016-08-12 09:54	Kitware Robot	Note Added: 0036738
2016-08-12 09:54	Kitware Robot	Status	expired => closed
2016-08-12 09:54	Kitware Robot	Resolution	open => moved
2016-08-12 09:54	Kitware Robot	Assigned To	=> Kitware Robot