[IGSTK-Developers] itk::mesh to igstk::meshobject

[Patrick Cheng]

Hi Sebastian,

I agree that we should implement the classes in a consistent fashion.

I have created a bug entry describing this issue:
http://public.kitware.com/Bug/view.php?id=6863

I have locally modified the code to use the event. If there is no 
objection, I will go ahead and commit the changes.

Thank you,

Patrick

sebastian ordas wrote:
> Hi Patrick,
> 
> I just wanted to say that if you were going to change the mesh reader, 
> an implementation similar to:
> 
>  "CTImageObserver" and "igstk::CTImageReader::ImageModifiedEvent()"
> 
> would be better than the current GetOutput() approach.
> 
> however, the current approach works fine, so I would give low priority 
> to this modification
> 
> but of course, I am not the right person to be asked about these things :-)
> 
> regards,
> Seba
> 
> 
> 
> On Mon, Apr 21, 2008 at 11:19 PM, Patrick Cheng 
> <cheng at isis.georgetown.edu <mailto:cheng at isis.georgetown.edu>> wrote:
> 
>     Hi Sebastian,
> 
>     Can you be more specific about the purpose of this event? Do you
>     want it to be loaded with MeshObject?
> 
>     Right now, the MeshObjectReader has a direct way to get the
>     MeshObject, using GetOutput(), which is not consistent with the
>     IGSTK's way of using event.
> 
>     We have the following options:
>     1. Do nothing.
>     2. Add event with loaded MeshObject and keep the GetOutput API
>     3. Add event with loaded MeshObject and remove the GetOutput API
> 
>     I prefer 2, what do your guys think? This will apply to other
>     SpatialObject classes as well.
> 
>     Thank you,
> 
>     Patrick
> 
>     sebastian ordas wrote:
> 
>         Hi Patrick,
> 
>         to my knowledge, there is no such "MeshModifiedEvent()" in the
>         igstk mesh reader. If so, this could be an opportunity for adding it
> 
>         regards,
>         Sebastian
> 
>         On Fri, Apr 18, 2008 at 6:25 PM, sebastian ordas
>         <sebastian.ordas at gmail.com <mailto:sebastian.ordas at gmail.com>
>         <mailto:sebastian.ordas at gmail.com
>         <mailto:sebastian.ordas at gmail.com>>> wrote:
> 
>            Hi Aswin John, you can check in the sandbox, the example
>            NeedleBIopsyNew: there you have the method:
>         RequestLoadTargetMesh()
>            let me know if you need to know how to convert a vtk/stl mesh
>         to a
>            mesh spatial object
> 
>            best,
>            sebastian
> 
> 
>