<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hi David,<div class=""><br class=""></div><div class="">When I remove the changes introduced in MR 856, the test does pass. All I need to do to cause the test to fail is uncomment “findEntity” (LoadJSON.cxx:868) (and the loops leading up to the call, but they don’t seem to effect the test), which is weird. Looking at Manager.cxx:923, findEntity is not really a const call? Passing trySessions=false to the findEntity call in LoadJSON.cxx:868 also seems to fix the problem. </div><div class=""><br class=""></div><div class="">Sincerely,</div><div class="">T.J.</div><div class=""><br class=""><div class="">
<div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;">Thomas J. Corona, Ph.D.<br class="">Kitware, Inc.<br class="">Senior R&D Engineer<br class="">21 Corporate Drive<br class="">Clifton Park, NY 12065-8662<br class="">Phone: 518-881-4443</div></div>

</div>
<br class=""><div><blockquote type="cite" class=""><div class="">On Sep 26, 2017, at 7:51 AM, David Thompson <<a href="mailto:david.thompson@kitware.com" class="">david.thompson@kitware.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">... Also, I've run valgrind on the test with no reported errors, so if it is stomping on memory, it's most probably happening with statically-allocated array bounds being exceeded.<br class=""><br class=""><span class="Apple-tab-span" style="white-space:pre">      </span>David<br class=""><br class=""><blockquote type="cite" class="">I've been trying to track down why discreteSmtkModelfrom2dmTest is failing on the dashboards.  The bug occurs because the discrete model's edge operator can't determine the ID of the point chosen for the split.<br class=""><br class="">The problem appears to be in smtk/bridge/discrete/operators/EdgeOperator.cxx:<br class=""><br class="">int EdgeOperator::convertToGlobalPointId(int localPid, vtkDiscreteModelEdge* cmbModelEdge)<br class=""><br class="">which is called with `localPid` set to a value from the operator's MeshSelectionItem. Inside convertToGlobalPointId, an error occurs because hasTessellation() returns a null pointer. The edge selected also has no mesh backing (i.e., edge.meshTessellation().isValid() is false).<br class=""><br class="">I am still confused as to why my recent change (SMTK MR 856) caused this to surface; all it does is stop a model file from being loaded twice -- and when it does, it prints out error messages that do not appear when running smtkModelfrom2dmTest.<br class=""><br class="">Does anyone who knows more about MeshSelectionItem, particularly what its integers index into, have a clue?<br class=""><br class=""><span class="Apple-tab-span" style="white-space:pre">   </span>Thanks,<br class=""><span class="Apple-tab-span" style="white-space:pre">        </span>David<br class=""></blockquote><br class=""></div></div></blockquote></div><br class=""></div></body></html>