[Insight-developers] WrapITK must found ImageCompare to run the
tests
Luis Ibanez
luis.ibanez at kitware.com
Thu Sep 7 07:48:15 EDT 2006
Hi Gaetan,
The C++ test drivers already have a built-in Image Compare
functionality. So, it seems that the problem is that it is not
available to the wrapping tests.
It was from the C++ test drivers that we took the code for creating
ImageCompare when we put it in the suggested material for the Insigth
Journal.
That being said, ImageCompare is such a basic tool, that I agree with
you that adding it to the default binaries to be built by ITK is a
good idea.
Option (3) also sounds like a good option, and it seems that it offers
extra advantages. My suggestion would be to do (1) right now, just
because it is so easy (we just need to commit the file), and then
proceed to also implement (3).
Would you like to undertake (3) ?
Luis
----------------------
Gaetan Lehmann wrote:
>
> Hi,
>
> To implement WrapITK's tests, I have used ImageCompare, which is not by
> default in ITK.
> There is several possible ways to go:
> (1) add ImageCompare to ITK
> (2) implement the image comparaison in the test drivers
> (3) implement a test driver in c++ usable for all the languages, and
> able to compare the images
>
> (1) is the simplest one, and would be much give ImageCompare by default
> to the user, so it would be easier to begin to prepare a contribution
> to the insight journal
> (2) will duplicate the image comparaison task - not so good
> (3) should be mainly an adaptation of the ITK test driver to set the
> environment needed to run the tests with the target languages. Also, I
> think that this test driver would be better for the insight journal
> than ImageCompare: we can run the test and validate its results in a
> single test; it is not possible with ImageCompare. Finally, it should
> be easy with this one to fix the lib path problem with java
>
> My preference goes to (3) - please let me know what you think about that
>
> Thanks,
>
> Gaetan
>
>
More information about the Insight-developers
mailing list