[Insight-developers] Adding Concept Chechking "float"/"double" to pixel type of GradientAnisotropicDiffusion

[Bill Lorensen]

Luis,

This change may cause backward compatibility problems. Old code may no
longer compile and the compiler error for the conept checking is cryptic on
some compilers. We should warn the user and give explicit instructions on
how to repair the code in the warning message.

I understand that the current code is not producing correct results. But
that is the fault of the itk development team and not the user of the code.
We should notify users in an instructive way on how to fix their code so
that it produces correct results. A cryptic compiler error is not
informative.

I'm sure others will want to comment on this.

Bill


On 8/15/07, Luis Ibanez <luis.ibanez at kitware.com> wrote:
>
>
> We just encountered in the mailing list that the
> GradientAnisotropic diffusion filter has to be
> instantiated over an image of "float/double" pixel
> type in order to perform computations properly.
>
> http://public.kitware.com/pipermail/insight-users/2007-August/023304.html
>
> If there are no objections I will add a ConceptCheck
> for this filter, and probably all the filters that
> use the PDE Solver.
>
> Otherwise they compile, run,
> and silently fail to produce a correct image as output.
>
>
>    Luis
>
>
> _______________________________________________
> Insight-developers mailing list
> Insight-developers at itk.org
> http://www.itk.org/mailman/listinfo/insight-developers
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://www.itk.org/mailman/private/insight-developers/attachments/20070815/29f426b9/attachment.htm