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

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

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


##########
opennlp-tools/src/main/java/opennlp/tools/ml/model/AbstractDataIndexer.java:
##########
@@ -188,17 +233,15 @@ protected List<ComparableEvent> index(ObjectStream<Event> 
events,
     return eventsToCompare;
   }
 
-  public int getNumEvents() {
-    return numEvents;
-  }
-
   /**
-   * Updates the set of predicated and counter with the specified event 
contexts and cutoff.
+   * Updates the set of predicates and counter with the specified event 
contexts and cutoff.
+   *
    * @param ec The contexts/features which occur in a event.
    * @param predicateSet The set of predicates which will be used for model 
building.
    * @param counter The predicate counters.
    * @param cutoff The cutoff which determines whether a predicate is included.
-   * @deprecated will be removed after 1.8.1 release
+   *
+   * @deprecated Use {{@link #update(String[], Map)}}. This method will be 
removed after 1.8.1 release

Review Comment:
   @rzo1 It seems worth documenting the removal on its own, for clarity in 
release notes. I'll open a separate Jira for this.





> Enhance JavaDoc in opennlp.tools.ml package
> -------------------------------------------
>
>                 Key: OPENNLP-1423
>                 URL: https://issues.apache.org/jira/browse/OPENNLP-1423
>             Project: OpenNLP
>          Issue Type: Improvement
>          Components: Machine Learning
>    Affects Versions: 2.1.0
>            Reporter: Martin Wiesner
>            Assignee: Martin Wiesner
>            Priority: Minor
>             Fix For: 2.1.1
>
>
> The JavaDoc of the _opennlp.tools.ml_ packages suffer from several 
> inconsistencies and missing descriptions. Moreover, several typos are present 
> that need sanitizing.
> The classes, interfaces, etc., require enhancements and/or additions to 
> provide more clarity for readers.



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

Reply via email to