[
https://issues.apache.org/jira/browse/OPENNLP-1410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17645765#comment-17645765
]
ASF GitHub Bot commented on OPENNLP-1410:
-----------------------------------------
kinow commented on code in PR #456:
URL: https://github.com/apache/opennlp/pull/456#discussion_r1045223051
##########
opennlp-tools/src/main/java/opennlp/tools/namefind/BioCodec.java:
##########
@@ -26,23 +26,39 @@
import opennlp.tools.util.SequenceCodec;
import opennlp.tools.util.Span;
+/**
+ * The default {@link SequenceCodec} implementation according to the {@code
BIO} scheme:
+ * <ul>
+ * <li>B: 'beginning' of a NE)</li>
+ * <li>I: 'inside', the word is inside a NE</li>
+ * <li>O: 'outside', the word is a regular word outside a NE)</li>
Review Comment:
Same here?
##########
opennlp-tools/src/main/java/opennlp/tools/namefind/BilouCodec.java:
##########
@@ -27,6 +27,23 @@
import opennlp.tools.util.SequenceValidator;
import opennlp.tools.util.Span;
+/**
+ * The default {@link SequenceCodec} implementation according to the {@code
BILOU} scheme.
+ * <ul>
+ * <li>B: 'beginning' of a NE)</li>
+ * <li>I: 'inside', the word is inside a NE)</li>
Review Comment:
I think the `NE)` has an extra `)` in these two cases?
> Enhance JavaDoc in opennlp.tools.namefind package
> -------------------------------------------------
>
> Key: OPENNLP-1410
> URL: https://issues.apache.org/jira/browse/OPENNLP-1410
> Project: OpenNLP
> Issue Type: Improvement
> Components: Name Finder
> Affects Versions: 2.1.0
> Reporter: Martin Wiesner
> Assignee: Martin Wiesner
> Priority: Minor
> Fix For: 2.1.1
>
>
> The JavaDoc the _opennlp.tools.namefind_ 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.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
