[
https://issues.apache.org/jira/browse/OPENNLP-1368?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17525024#comment-17525024
]
ASF GitHub Bot commented on OPENNLP-1368:
-----------------------------------------
jzonthemtn commented on code in PR #415:
URL: https://github.com/apache/opennlp/pull/415#discussion_r854176140
##########
opennlp-tools/src/main/java/opennlp/tools/util/model/BaseModel.java:
##########
@@ -416,10 +416,8 @@ protected void validateArtifactMap() throws
InvalidFormatException {
// Version check is only performed if current version is not the
dev/debug version
if (!Version.currentVersion().equals(Version.DEV_VERSION)) {
- // Major and minor version must match, revision might be
- // this check allows for the use of models of n minor release behind
current minor release
- if (Version.currentVersion().getMajor() != version.getMajor() ||
- Version.currentVersion().getMinor() - 4 > version.getMinor()) {
+ // Support OpenNLP 1.x models.
+ if (version.getMajor() != 1) {
Review Comment:
Since we're going to be releasing OpenNLP 2.0 we can't just compare the
major version numbers. So replacing the check on major version numbers with a
check to just make sure the model file is version 1.x.
> Allow OpenNLP 2.x to use OpenNLP 1.x models
> -------------------------------------------
>
> Key: OPENNLP-1368
> URL: https://issues.apache.org/jira/browse/OPENNLP-1368
> Project: OpenNLP
> Issue Type: Task
> Components: Machine Learning
> Affects Versions: 2.0.0
> Reporter: Jeff Zemerick
> Assignee: Jeff Zemerick
> Priority: Major
>
> Allow OpenNLP 2.x to use OpenNLP 1.x models
--
This message was sent by Atlassian Jira
(v8.20.7#820007)
