[Insight-developers] Re: MattesMutualInformation

[Luis Ibanez]

Lydia,

The need for the SetFixedImageRegion() was to allow the FEM
framework to operate on small regions around the nodes.

It seems in the majority of cases it will be just fine to
set this region to the buffered region taken from the fixed
image.

For the regresion tests it will probably be simpler to
continue just testing the Transform parameters using some
level of tolerance.



    Luis


----------------------

Lydia Ng wrote:
> 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
> 
>