[ 
https://issues.apache.org/jira/browse/OPENNLP-1404?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17643026#comment-17643026
 ] 

ASF GitHub Bot commented on OPENNLP-1404:
-----------------------------------------

mawiesne commented on code in PR #446:
URL: https://github.com/apache/opennlp/pull/446#discussion_r1038990369


##########
opennlp-tools/src/main/java/opennlp/tools/postag/POSTaggerFactory.java:
##########
@@ -95,8 +83,19 @@ public POSTaggerFactory(byte[] featureGeneratorBytes, final 
Map<String, Object>
     this.posDictionary = posDictionary;
   }
 
+
+  // reduced visibility to ensure deprecation is respected in future versions
+  @Deprecated
+  POSTaggerFactory(Dictionary ngramDictionary, TagDictionary posDictionary) {
+    this.init(ngramDictionary, posDictionary);
+
+    // TODO: This could be made functional by creating some default feature 
generation
+    // which uses the dictionary ...

Review Comment:
   This comment was there previously. Maybe, I just moved it. So no actual 
change intended here. 





> Enhance JavaDoc in opennlp.tools.postag package
> -----------------------------------------------
>
>                 Key: OPENNLP-1404
>                 URL: https://issues.apache.org/jira/browse/OPENNLP-1404
>             Project: OpenNLP
>          Issue Type: Improvement
>          Components: Documentation, POS Tagger
>    Affects Versions: 2.1.0
>            Reporter: Martin Wiesner
>            Priority: Minor
>             Fix For: 2.1.1
>
>
> The JavaDoc of the _opennlp.tools.postag_ package suffers from several 
> inconsistencies and missing descriptions. Moreover, several typos are present 
> that need sanitizing.
> It needs enhancements and/or additions to provide more clarity for readers of 
> this part of the OpenNLP API.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to