jzonthemtn commented on a change in pull request #377:
URL: https://github.com/apache/opennlp/pull/377#discussion_r436409417
##########
File path:
opennlp-tools/src/test/java/opennlp/tools/eval/SourceForgeModelEval.java
##########
@@ -410,7 +410,7 @@ public void evalParserModel() throws Exception {
}
}
- Assert.assertEquals(new
BigInteger("312218841713337505306598301082074515847"),
+ Assert.assertEquals(new
BigInteger("68039262350771988792233880373220954061"),
Review comment:
@kinow This number changed a lot because it is an MD5 hash value. Looks
like the test is using the parser chunking model to parse the
`leipzig/eng_news_2010_300K-sentences.txt` input file. At the end it computes
the hash of the parse string and compares it to an expected value.
`StrictMath` must be causing a difference that's visible in the hash value.
I'm guessing it's probably like those other tests where it's a small rounding
difference somewhere.
I did verify that `68039262350771988792233880373220954061` is the same value
on Java 8 and Java 11.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
