<div>Hi Bill,</div>I had noticed this. I have been assuming that Marcus or someone has been gradually reintroducing the tests. <div>If this is not the case is there something I should do?</div><div><br></div><div>Andrew<br>
<br><div class="gmail_quote">On Wed, May 16, 2012 at 9:12 AM, Bill Lorensen <span dir="ltr"><<a href="mailto:bill.lorensen@gmail.com" target="_blank">bill.lorensen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Andrew,<br>
<br>
Looks to me like the Parametric classes are not being tested by VTK6<br>
<br>
Bill<br>
<div class="HOEnZb"><div class="h5"><br>
On Tue, May 15, 2012 at 6:47 PM, Andrew Maclean<br>
<<a href="mailto:andrew.amaclean@gmail.com">andrew.amaclean@gmail.com</a>> wrote:<br>
> Hi Bill, David,<br>
>   As one of you pointed out earlier, the VTK Parametric classes will be<br>
> affected by this change. In particular, the regression tests will<br>
> fail due to the higher precision of Pi. I would only expect that the<br>
> pictures for the regression tests would need to be updated.<br>
><br>
> When I wrote these in conjunction with Will, I must admit I was frustrated<br>
> that vtkMath::Pi only returned float. However I made a decision to use<br>
> vtkMath::Pi rather than yet another constant for Pi in the hopes that this<br>
> would change in the future. The greater precision should improve the look of<br>
> some of the surfaces. E.g. the little imperfection at the pole of the<br>
> ellipsoid should get smaller.<br>
><br>
> Let me know if I can help in any way.<br>
><br>
> When things settle, I will also update the PDF titled "Parametric Equations<br>
> for Surfaces" in the documentation to VTK 6.<br>
><br>
> Regards<br>
>    Andrew<br>
><br>
><br>
><br>
>><br>
>><br>
>> ---------- Forwarded message ----------<br>
>> From: David Gobbi <<a href="mailto:david.gobbi@gmail.com">david.gobbi@gmail.com</a>><br>
>> To: Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>><br>
>> Cc: VTK Developers <<a href="mailto:vtk-developers@vtk.org">vtk-developers@vtk.org</a>><br>
>> Date: Mon, 14 May 2012 16:38:37 -0600<br>
>> Subject: Re: [vtk-developers] DoublePi proposal<br>
>> Bill, the main reason that I put my patch on hold was that I wanted to<br>
>> wait until VTK 6 had reached at least the same level of test coverage<br>
>> as VTK 5.  I still think that this is a valid reason for keeping this<br>
>> patch on hold.  The added pi precision will cause subtle changes in<br>
>> many regression images, and some of the changes will be significant<br>
>> enough to cause test failures.  That's great if all the regression<br>
>> tests are running, because we can examine any test failures that occur<br>
>> and replace the regression images as necessary.<br>
>><br>
>> But right now in VTK 6, about half of our regression tests aren't<br>
>> running.  So if this Pi patch is pushed now, we won't see all the<br>
>> regression test failures immediately.  We'll see some regression test<br>
>> failures some weeks (or months?) later as the various tests are<br>
>> re-activated, and we won't be sure if the failures were caused by this<br>
>> patch, or if they were caused by some other patch.<br>
>><br>
>>  - David<br>
>><br>
>><br>
>> On Mon, May 14, 2012 at 11:59 AM, David Gobbi <<a href="mailto:david.gobbi@gmail.com">david.gobbi@gmail.com</a>><br>
>> wrote:<br>
>> > <a href="http://review.source.kitware.com/#/t/427/" target="_blank">http://review.source.kitware.com/#/t/427/</a><br>
>> ><br>
>> ><br>
>> > On Mon, May 14, 2012 at 10:50 AM, Bill Lorensen<br>
>> > <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>> wrote:<br>
>> >> David,<br>
>> >><br>
>> >> Can you point me to the gerrit topic? If not, I can start from scratch.<br>
>> >><br>
>> >> Bill<br>
><br>
> --<br>
> ___________________________________________<br>
> Andrew J. P. Maclean<br>
><br>
> ___________________________________________<br>
<br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Unpaid intern in BillsBasement at noware dot com<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br>___________________________________________<br>Andrew J. P. Maclean<br><br>___________________________________________<br>
</div>