<div dir="ltr">Can you tell us more about the failure conditions?<div><br></div><div>I strongly suspect the code is correct. git blame indicates that it has been doing this scaling and biasing with minor variations since 2000.</div><div><br></div><div>Try running the tests locally with your proposed change. Test failures should indicate use cases of the existing code path.</div><div><br></div><div><br></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>David E DeMarle<br>Kitware, Inc.<br>Principal Engineer<br>21 Corporate Drive<br>Clifton Park, NY 12065-8662<br>Phone: 518-881-4909</div></div></div></div></div></div>
<br><div class="gmail_quote">On Fri, May 25, 2018 at 6:27 AM, xVict <span dir="ltr"><<a href="mailto:suharev@roentgenprom.ru" target="_blank">suharev@roentgenprom.ru</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi All!<br>
When vtkImageMapToWindowLevelColors  using lookupTable,<br>
in the method: <br>
template <class T><br>
void vtkImageMapToWindowLevelColors<wbr>Execute(<br>
  vtkImageMapToWindowLevelColors *self,<br>
  vtkImageData *inData, T *inPtr,<br>
  vtkImageData *outData,<br>
  unsigned char *outPtr,<br>
  int outExt[6], int id)<br>
<br>
in the condition ( line: 323 in master from (<42f7955511> 2018-05-23 [David<br>
E. DeMarle])):<br>
<br>
if ( lookupTable )<br>
      {<br>
        lookupTable-><wbr>MapScalarsThroughTable2(<br>
          inPtr1,<br>
          static_cast<unsigned char *>(outPtr1),<br>
          dataType,extX,<wbr>numberOfComponents,<br>
          outputFormat);<br>
<br>
        for (idxX = 0; idxX < extX; idxX++)<br>
<br>
the for-loop is not needed. It spoils the rgb values expected from the<br>
lookupTable.<br>
I don't understand:<br>
<br>
          *optr = static_cast<unsigned char>((*optr * ushort_val) >> 8);<br>
<br>
gray = linear conversion of values from the specified window level to range<br>
0-255<br>
<br>
r_out = r * gray / 256<br>
g_out = g * gray / 256<br>
b_out = b * gray / 256<br>
<br>
I propose to remove this for-loop. I.e., lines: 331 - 367.<br>
After it for example user's lookuptable correctly applied for imageViwer2.<br>
<br>
Sincerely, Victor.<br>
<br>
<br>
<br>
--<br>
Sent from: <a href="http://vtk.1045678.n5.nabble.com/VTK-Dev-f1251487.html" rel="noreferrer" target="_blank">http://vtk.1045678.n5.nabble.<wbr>com/VTK-Dev-f1251487.html</a><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="https://vtk.org/mailman/listinfo/vtk-developers" rel="noreferrer" target="_blank">https://vtk.org/mailman/<wbr>listinfo/vtk-developers</a><br>
<br>
</blockquote></div><br></div>