<div dir="ltr"><div dir="ltr">On Mon, Mar 11, 2019 at 10:46 AM Matt Brown via Kwiver-users <<a href="mailto:kwiver-users@public.kitware.com" target="_blank">kwiver-users@public.kitware.com</a>> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">intersection() could throw a warning that one of the boxes is empty, but it should still return 0 intersection area, I would think. </div></blockquote><div>The issue is, intersection() returns a bounding box, not an area. So there isn't a clear ideal way to return a bounding box that properly encodes "empty." Right now, it returns a bounding box with all properties populated in an undefined manner.<br></div></div></div></blockquote></div><div><br></div><div dir="ltr">vgl_intersection returns an empty box, which is always min(x,y)=1, max(x,y)=0; vgl_box_2d's implementation of is_empty() returns true whenever the min coordinates are less than the max.<br></div><div dir="ltr"><br></div><div dir="ltr">Given how often vgl_intersection appears in our legacy code (probably other vgl code as well), would it make sense to copy over the tests from vgl to make sure the semantics are the same?</div><div dir="ltr"><br><div>-Roddy</div><div><br></div></div></div></div>