[Insight-developers] Moving ChanAndVese out of Review ?

[Arnaud GELAS]

Luis,

In short term, it makes sense to move this code out of the review directory!

However, we are in the process of refactoring the whole framework, and 
we are in the process of drastically change the whole hierarchy. Note, 
as of now we use Chan and Vese as first example to test, and validate 
our development.

So in the long term, I am not sure if it really makes sense to extract 
all these classes from the Review directory. But some of these classes 
will be used in the future version, and that's the reason why, I have 
been submitting this patch to gerrit:

     http://review.source.kitware.com/#change,1460

If needed, we can discuss about it in an appropriate TConf.

Thanks,
Arnaud

On 04/25/2011 02:43 PM, Luis Ibanez wrote:
> Hi Kishore, Arnaud,
>
> As our initial work in the Code Review effort
> we are trying to clean up the dependencies
> on the Review directory.
>
> Currently one of them is the use of ChanAndVese
> filters in the Examples.
>
> We would like to do a full code review of the
> ChanAndVese classes and (once they pass the
> code review) move them out of the Review directory
> to put them in their final module in the toolkit
>
> Most likely the LevelSets modules...
>
> Would you have any concerns with
> us fixing and moving this code ?
>
> Were you anticipating any major refactoring
> of these classes ?
>
>
>      Thanks
>
>
>           Luis