[Insight-developers] Failing Test after check in

[Hans Johnson]

Bill,

You are awesome!  Thanks for addressing this issue.

It looks like we are nearing a green dashboard again!

Hans

On 12/18/08 12:22 PM, "Bill Lorensen" <bill.lorensen at gmail.com> wrote:

> Hans,
> 
> I fixed the problem. It was a tricky one. I recall that if an ITK
> filter is reused and produces a different sized output, its request
> region is not correct. I fixed the pyramid code to
> UpdateLargestPossibleRegion() on the resample filter. Also, the
> resample filter had a bug if the OutputStartIndex was not 0. The
> itkMultiResolutionPDEDeformableRegistrationTest created an image with
> a non-zero index. That caused the problem to surface in the resample
> filter.
> 
> Bill
> 
> On Wed, Dec 17, 2008 at 2:00 PM, Bill Lorensen <bill.lorensen at gmail.com>
> wrote:
>> The other failures were different I think. I made this test into a
>> regression test. The older failures were regression failures I
>> believe.
>> 
>> BTW, I'm almost ready to checkin the ShrinkFilter fixes.
>> 
>> Bill
>> 
>> On Wed, Dec 17, 2008 at 1:52 PM, Hans Johnson <hans-johnson at uiowa.edu> wrote:
>>> Bill,
>>> 
>>> This has been failing since at least December 15th nightly builds.  I
>>> committed my code last night at 9:30pm.
>>> 
>>> I don't think that I am the originating culprit in this failure.  With that
>>> said, I may be compounding the problem with my bug fixes as this filter
>>> almost certainly uses the itkMultiResolutionPyramidImageFilter.
>>> 
>>> 
>>> Hans
>>> 
>>> 
>>> 
>>> 
>>> On 12/17/08 11:47 AM, "Bill Lorensen" <bill.lorensen at gmail.com> wrote:
>>> 
>>>> Hans,
>>>> 
>>>> The itkMultiResolutionPDEDeformableRegistrationTest is failing on some
>>>> platforms after your recent checkins.
>>>> http://www.cdash.org/CDash/testSummary.php?project=2&name=itkMultiResolutio
>>>> nPD
>>>> EDeformableRegistrationTest&date=2008-12-17
>>>> 
>>>> I think they are all failing in a similar way. If I recall correctly,
>>>> this test uses images that have a non-zero starting index. This may be
>>>> causing the problem.
>>>> 
>>>> Bill
>>> 
>>> 
>>> 
>>> Notice: This UI Health Care e-mail (including attachments) is covered by the
>>> Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is confidential
>>> and may be legally privileged.  If you are not the intended recipient, you
>>> are hereby notified that any retention, dissemination, distribution, or
>>> copying of this communication is strictly prohibited.  Please reply to the
>>> sender that you have received the message in error, then delete it.  Thank
>>> you.
>>> 
>>> 
>>> 
>>