Fixed -unkWord option in RNNOptions.java

[mschnoell]

The -unkWord option is usable now ;-)

[AngledLuffa]

Oops, how embarrassing. Thanks for reporting that!

We really should have an arg parsing library (ideally one which can handle
enums)

On Thu, Dec 17, 2015 at 8:05 AM, Martin [email protected] wrote:

The -unkWord option is usable now ;-)

You can view, comment on, or merge this pull request online at:

#115
Commit Summary

• Merge pull request Could the project switch to using log4j for logs? #1 from stanfordnlp/master
• Fixed -unkWord option in RNNOptions.java

File Changes

• M src/edu/stanford/nlp/sentiment/RNNOptions.java
  https://github.com/stanfordnlp/CoreNLP/pull/115/files#diff-0 (2)

Patch Links:

• https://github.com/stanfordnlp/CoreNLP/pull/115.patch
• https://github.com/stanfordnlp/CoreNLP/pull/115.diff

—
Reply to this email directly or view it on GitHub
#115.

[gangeli]

Technically, we do have an arg parsing library (http://nlp.stanford.edu/nlp/javadoc/javanlp/), and it even handles Enums. I'm just bad at advertising it...

[AngledLuffa]

What's it called? It doesn't show up on that page, at least not if you
search for "arg"

On Thu, Dec 17, 2015 at 2:18 PM, Gabor Angeli [email protected]
wrote:

Technically, we do have an arg parsing library (
http://nlp.stanford.edu/nlp/javadoc/javanlp/), and it even handles Enums.
I'm just bad at advertising it...

—
Reply to this email directly or view it on GitHub
#115 (comment).

[gangeli]

Ack, I forgot Javadoc has iframes: http://nlp.stanford.edu/nlp/javadoc/javanlp-3.6.0/edu/stanford/nlp/util/Execution.html. The OpenIE code uses it, for example: https://github.com/stanfordnlp/CoreNLP/blob/master/src/edu/stanford/nlp/naturalli/OpenIE.java#L99