<div dir="ltr"><div>Bill,</div><div><br></div><div>I would be surprised if it has anything to do with BORDER_PIXEL. BORDER_PIXEL comes into play only when `FixBoundary` is ON, and it's OFF by default and tiling is used -- which is rarely, if at all used for VTK test baselines. It should be deprecated as it's no longer needed. It was needed in OpenGL1 days for certain OpenGL drivers where when rendering wireframe, the wireframe for a cell would get artificially closed if it intersected the viewport boundary. That doesn't happen with OpenGL2.</div><div><br></div><div>Utkarsh</div><div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 5, 2017 at 10:59 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Folks,<br>
<br>
Interesting.<br>
<br>
When an image is passed through vtkWindowToImageFilter, the size of<br>
the output image is reduced by 1 in each dimension.<br>
<br>
So if you SetSize on a render window,<br>
renderWindow->SetSize(n, m), the saved size will be n-1, m-1.<br>
<br>
This has to do with the setting in vtkWindowTiImageFilter:<br>
#define BORDER_PIXEL 2<br>
<br>
I'm not sure of the logic using BORDER_PIXEL, but if I define it as 0,<br>
the saved size matches the SetSize.<br>
<br>
We can't change vtkWindowToImageFilter since it would break every<br>
regression test.<br>
<br>
I can fix the odd baseline issue by setting sizes in the failing tests<br>
that take into account the logic in window to image filter.<br>
<br>
Bill<br>
<br>
<br>
On Mon, Jul 3, 2017 at 2:52 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>> wrote:<br>
> There are only 23 tests that have odd sizes and they are all python<br>
> tests, oddly enough.<br>
><br>
> I'll change them to have even sizes and I'll update the baselines. For<br>
> those tests, I'll delete all baseline versions before I add the new<br>
> one.<br>
><br>
> Bill<br>
><br>
><br>
> On Mon, Jul 3, 2017 at 12:54 PM, Sean McBride <<a href="mailto:sean@rogue-research.com">sean@rogue-research.com</a>> wrote:<br>
>> On Fri, 30 Jun 2017 09:56:41 -0400, Shawn Waldon said:<br>
>><br>
>>>Ping?  Is anyone still working on this?<br>
>><br>
>> I'm not.  I don't believe we agreed on a plan either.<br>
>><br>
>> I've asked on the cocoa-dev list and am pretty sure we are stuck with NSWindow's behaviour, which is: round sizes *up* to even integral sizes.<br>
>><br>
>> David's suggestion of changing the testing system so that it tolerates a 1-pixel difference in the size in the images I agree is probably best, in the short term at least.  Still, if we were starting from scratch, it seems to me that having test output and baselines of identical sizes would be preferable.  Perhaps as tests are maintained, they should be updated to not use odd sizes?<br>
>><br>
>> I wonder too if we ought to add a log/warning to SetSize() implementations when users of the API provide odd sizes?<br>
>><br>
>> Sean<br>
>><br>
>><br>
>> ______________________________<wbr>_________________<br>
>> Powered by <a href="http://www.kitware.com" rel="noreferrer" target="_blank">www.kitware.com</a><br>
>><br>
>> Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer" target="_blank">http://www.kitware.com/<wbr>opensource/opensource.html</a><br>
>><br>
>> Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" rel="noreferrer" target="_blank">http://markmail.org/search/?q=<wbr>vtk-developers</a><br>
>><br>
>> Follow this link to subscribe/unsubscribe:<br>
>> <a href="http://public.kitware.com/mailman/listinfo/vtk-developers" rel="noreferrer" target="_blank">http://public.kitware.com/<wbr>mailman/listinfo/vtk-<wbr>developers</a><br>
>><br>
><br>
><br>
<span class="gmail-HOEnZb"><font color="#888888">><br>
> --<br>
> Unpaid intern in BillsBasement at noware dot com<br>
<br>
<br>
<br>
--<br>
Unpaid intern in BillsBasement at noware dot com<br>
______________________________<wbr>_________________<br>
Powered by <a href="http://www.kitware.com" rel="noreferrer" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer" target="_blank">http://www.kitware.com/<wbr>opensource/opensource.html</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" rel="noreferrer" target="_blank">http://markmail.org/search/?q=<wbr>vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtk-developers" rel="noreferrer" target="_blank">http://public.kitware.com/<wbr>mailman/listinfo/vtk-<wbr>developers</a><br>
<br>
</font></span></blockquote></div><br></div></div></div>