[Insight-developers] RE: MattesMutualInformation

Lorensen, William E (Research) lorensen@crd.ge.com
Tue, 1 Apr 2003 16:11:39 -0500


Lydia,
I have added the capability to have multiple baseline images for a given
test. I'm out of town now. More details to followed.

Bill

-----Original Message-----
From: Lydia Ng [mailto:lng@insightful.com]
Sent: Tuesday, April 01, 2003 11:59 AM
To: Lorensen, William E (Research); Insight-developers (E-mail)
Cc: Luis.ibanez@kitware.com
Subject: RE: MattesMutualInformation


Bill, Luis:

The ABR and IPR were related to too large a FixedImageRegion given to
metric. I have put in extra protection by always cropping the
FixedImageRegion to be within the buffered region of the FixedImage.

The next step is to figure out the problem in the region computation
inside the MultiResolutionRegistrationMethod.
Luis: is there any reason why we have to compute the regions at all and
not just ask the downsampled image for it since the
MultiResolutionRegistrationMethod only performs registration on the
whole images?

My change has got rid of the purify error. However the regression test
is not passing. In the tests even with simple test images, all the
platforms converge to slightly different values. Hence, the images
produced by the resampling would be slightly different. It will be
tricky to do the image comparison. In the tests, I have been just
checking that the parameters are within some range to what I expect.

- Lydia

> -----Original Message-----
> From: Lorensen, William E (Research) [mailto:lorensen@crd.ge.com]
> Sent: Sunday, March 30, 2003 8:16 AM
> To: Lydia Ng; Insight-developers (E-mail)
> Subject: MattesMutualInformation
> 
> Lydia,
> I added a couple of new regression tests to Examples/Registration
> yesterday.
> I noticed that the MultiRes examples were crashing on the Borland
build.
> Today, purify is pointing out Array Bound Reads and Invalid Pointer
> References on the two MultiRes examples.
> 
> I took a quick look to see if it was an "easy" fix. Too hard for me on
a
> Sunady morning.
> 
> Perhaps you can take a look.
> 
> Bill