Hi Luis, <br><br>I'll get started on it. <br><br>Cheers, <br><br>Michel<br><br><div class="gmail_quote">On Fri, May 22, 2009 at 9:36 AM, Luis Ibanez <span dir="ltr"><<a href="mailto:luis.ibanez@kitware.com">luis.ibanez@kitware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
Hi Michel,<br>
<br>
You may actually find useful to write the test that Bill suggests.<br>
It may help solve some of the remaining failing tests.<br>
<br>
It all depends on how close you feel you are to solve the remaining<br>
issues.<br>
<br>
<br>
Thanks,<br>
<br>
<br>
Luis<br>
<br>
<br>
-------------------<br>
Michel Audette wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="im">
Hi gents,<br>
<br>
I can certainly write a test that implements what Bill was proposing. For the time being, my attention is still focused on the failing tests on the Dashboard, but assuming that Luis is okay with this, we can tackle this test shortly.<br>
<br>
Best wishes,<br>
<br>
Michel<br>
<br></div><div class="im">
On Thu, May 21, 2009 at 9:05 AM, Wes Turner <<a href="mailto:wes.turner@kitware.com" target="_blank">wes.turner@kitware.com</a> <mailto:<a href="mailto:wes.turner@kitware.com" target="_blank">wes.turner@kitware.com</a>>> wrote:<br>
<br>
I think a warning makes sense until we have better validation. <br>
Michel/Luis is this something that fits in with what you are doing?<br>
<br>
- Wes<br>
<br>
<br>
On Thu, May 21, 2009 at 8:37 AM, Bill Lorensen<br></div><div class="im">
<<a href="mailto:bill.lorensen@gmail.com" target="_blank">bill.lorensen@gmail.com</a> <mailto:<a href="mailto:bill.lorensen@gmail.com" target="_blank">bill.lorensen@gmail.com</a>>> wrote:<br>
<br>
Simon,<br>
<br>
My point is that we have not validated the new code as far as I<br>
can tell.<br>
<br>
Bill<br>
<br>
On Thu, May 21, 2009 at 7:57 AM, Simon Warfield<br>
<<a href="mailto:simon.warfield@childrens.harvard.edu" target="_blank">simon.warfield@childrens.harvard.edu</a><br></div><div><div></div><div class="h5">
<mailto:<a href="mailto:simon.warfield@childrens.harvard.edu" target="_blank">simon.warfield@childrens.harvard.edu</a>>> wrote:<br>
> Bill Lorensen wrote:<br>
>><br>
>> Yes, I think that will be OK. We should add an Attention:<br>
warning to<br>
>> the CMakeLists.txt file. Something like:<br>
>><br>
>> SET(msg "Attention: You have chosen to enable the use of<br>
>> cell-centered coordinates.")<br>
>><br>
><br>
> The tool kit currently uses a mixture of both, so the above<br>
is a bit<br>
> misleading. How about instead:<br>
> SET(msg "Attention: You have chosen to enable the consistent<br>
use of centered<br>
> pixel coordinates.")<br>
><br>
> We should explain the pros and cons of the choice e.g.:<br>
>><br>
>> SET(msg "${msg} This new functionality has not been fully<br>
validated.<br>
>> USE AT YOUR OWN RISK.")<br>
>><br>
><br>
> SET(msg "${msg} The old functionality creates an inconsistent<br>
physical<br>
> coordinate system.")<br>
>><br>
>> SET(msg "${msg} With this ON, you can expect to see<br>
differences in<br>
>> registration and interpolation results.")<br>
>><br>
><br>
> differences -> improvements<br>
>><br>
>> # display the message during the setup<br>
>> MESSAGE("${msg}")<br>
>><br>
><br>
> --<br>
> Simon<br>
>><br>
>> On Wed, May 20, 2009 at 11:23 PM, Wes Turner<br>
</div></div></blockquote>
</blockquote></div><br><br clear="all"><br>-- <br>Michel Audette, Ph.D. <br>R & D Engineer, <br>Kitware Inc.,<br>Chapel Hill, N.C. <br><br>