<div dir="ltr">Hi Christoph<div><br></div><div>I've added this as an issue in github: <a href="https://github.com/biojava/biojava/issues/355">https://github.com/biojava/biojava/issues/355</a></div><div><br></div><div>As I say in there, the first issue you report does work for me. The second I can reproduce. Let's continue the discussion over in github in any case.</div><div><br></div><div>Jose</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 16, 2015 at 7:45 AM, Christopher Gillies <span dir="ltr"><<a href="mailto:cgillies@umich.edu" target="_blank">cgillies@umich.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi biojava,<br>
<br>
I am using version 4.1.0 and I am getting some unexpected behavior from the Location class.<br>
<br>
Please see the unit test below. The union operator returns 51227381 as the bioEnd() but it should be 51227382.<br>
<br>
Location l1 = Location.fromBio(51227320, 51227381, '+');<br>
Location l2 = Location.fromBio(51227323, 51227382, '+');<br>
<br>
Location union = l1.union(l2);<br>
assertEquals(51227320,union.bioStart());<br>
assertEquals(51227382,union.bioEnd());<br>
<br>
<br>
<br>
<br>
The documentation for the intersection method says that it will return null when there is no overlap for two locations, but the code appears to fail for boundary conditions. The following unit test fails. Shouldn’t the intersection method return null for this test case?<br>
<br>
Location l1 = Location.fromBio(100, 200, '+');<br>
Location l2 = Location.fromBio(1, 99, '+');<br>
Location intersection = l1.intersection(l2);<br>
assertNull(intersection);<br>
<br>
Thanks,<br>
<br>
Chris<br>
_______________________________________________<br>
Biojava-l mailing list  -  <a href="mailto:Biojava-l@mailman.open-bio.org">Biojava-l@mailman.open-bio.org</a><br>
<a href="http://mailman.open-bio.org/mailman/listinfo/biojava-l" rel="noreferrer" target="_blank">http://mailman.open-bio.org/mailman/listinfo/biojava-l</a></blockquote></div><br></div>