[jira] [Updated] (OPENNLP-1010) NaiveBayes serialized models are not working

2017-04-07 Thread William Colen (JIRA)

 [ 
https://issues.apache.org/jira/browse/OPENNLP-1010?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

William Colen updated OPENNLP-1010:
---
Description: 
The behaviour of a serialized model is different from a model that was never 
serialized. For example, sometimes using the serialized we get only one 
outcome, while it was working perfectly during cross validation (when we use it 
before serializing).
The attached test reproduces it. It compares the outcome array of a model that 
was never serialized and the same model that was serialized and loaded.

  was:
Behaviour of the serialized model is different of the was never serialized. For 
example, sometimes we get only one outcome, always.
The attached test reproduces it. It compares the outcome array of the model 
that was never serialized and one that was loaded from serialized.


> NaiveBayes serialized models are not working
> 
>
> Key: OPENNLP-1010
> URL: https://issues.apache.org/jira/browse/OPENNLP-1010
> Project: OpenNLP
>  Issue Type: Bug
>  Components: Machine Learning
>Affects Versions: 1.7.2
>Reporter: William Colen
> Attachments: NaiveBayesSerializedCorrectnessTest.java
>
>
> The behaviour of a serialized model is different from a model that was never 
> serialized. For example, sometimes using the serialized we get only one 
> outcome, while it was working perfectly during cross validation (when we use 
> it before serializing).
> The attached test reproduces it. It compares the outcome array of a model 
> that was never serialized and the same model that was serialized and loaded.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (OPENNLP-1015) Add tests for DataIndexers

2017-04-07 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on OPENNLP-1015:
-

GitHub user kojisekig opened a pull request:

https://github.com/apache/opennlp/pull/152

OPENNLP-1015: Add tests for DataIndexers

Thank you for contributing to Apache OpenNLP.

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

### For all changes:
- [Y] Is there a JIRA ticket associated with this PR? Is it referenced 
 in the commit message?

- [Y] Does your PR title start with OPENNLP- where  is the JIRA 
number you are trying to resolve? Pay particular attention to the hyphen "-" 
character.

- [Y] Has your PR been rebased against the latest commit within the target 
branch (typically master)?

- [Y] Is your initial contribution a single, squashed commit?

### For code changes:
- [Y] Have you ensured that the full suite of tests is executed via mvn 
clean install at the root opennlp folder?
- [Y] Have you written or updated unit tests to verify your changes?
- [N/A] If adding new dependencies to the code, are these dependencies 
licensed in a way that is compatible for inclusion under [ASF 
2.0](http://www.apache.org/legal/resolved.html#category-a)? 
- [N/A] If applicable, have you updated the LICENSE file, including the 
main LICENSE file in opennlp folder?
- [N/A] If applicable, have you updated the NOTICE file, including the main 
NOTICE file found in opennlp folder?

### For documentation related changes:
- [N/A] Have you ensured that format looks appropriate for the output in 
which it is rendered?

### Note:
Please ensure that once the PR is submitted, you check travis-ci for build 
issues and submit an update to your PR as soon as possible.


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/kojisekig/opennlp dataindexer-test

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/opennlp/pull/152.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #152


commit f0622f888ae44d18c4dc4c37f3414677d648f5c2
Author: koji 
Date:   2017-04-07T12:50:02Z

OPENNLP-1015: Add tests for DataIndexers




> Add tests for DataIndexers
> --
>
> Key: OPENNLP-1015
> URL: https://issues.apache.org/jira/browse/OPENNLP-1015
> Project: OpenNLP
>  Issue Type: Test
>  Components: Build, Packaging and Test, Name Finder
>Reporter: Koji Sekiguchi
>Priority: Minor
>  Labels: test
>
> Seeing DataIndexers, I thought we could refactor them. However, when I tried 
> to do it, I realized there is no tests for DataIndexers.
> Before refactoring, they should have their tests. I'll post PR soon.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Created] (OPENNLP-1015) Add tests for DataIndexers

2017-04-07 Thread Koji Sekiguchi (JIRA)
Koji Sekiguchi created OPENNLP-1015:
---

 Summary: Add tests for DataIndexers
 Key: OPENNLP-1015
 URL: https://issues.apache.org/jira/browse/OPENNLP-1015
 Project: OpenNLP
  Issue Type: Test
  Components: Build, Packaging and Test, Name Finder
Reporter: Koji Sekiguchi
Priority: Minor


Seeing DataIndexers, I thought we could refactor them. However, when I tried to 
do it, I realized there is no tests for DataIndexers.

Before refactoring, they should have their tests. I'll post PR soon.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Updated] (OPENNLP-6) Create a new OpenNLP project logo for the site

2017-04-07 Thread Bruno P. Kinoshita (JIRA)

 [ 
https://issues.apache.org/jira/browse/OPENNLP-6?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Bruno P. Kinoshita updated OPENNLP-6:
-
Attachment: koji-kinoshita-tomasso-02.png

Uploading a few variations, with

* No gradient
* Different colours used for the book and for the letters
* Some font variations
* Inverted book, so that the writer is right handed (for some reason it looked 
more natural that way for me)

We can call these examples 1 - 6. So if you would like to see a combination of 
something from two or three variations, just name them that way :-) (i.e. get 
the colours from number #1, and the font from number #4, and so it goes)

> Create a new OpenNLP project logo for the site
> --
>
> Key: OPENNLP-6
> URL: https://issues.apache.org/jira/browse/OPENNLP-6
> Project: OpenNLP
>  Issue Type: Improvement
>  Components: Website
>Reporter: Joern Kottmann
>Assignee: Joern Kottmann
>Priority: Minor
>  Labels: help-wanted
> Fix For: 1.8.0
>
> Attachments: kinow-opennlp-1.png, kinow-opennlp-2.png, 
> kinow-opennlp-3.png, kinow-opennlp-3-variations.png, koji-kinoshita-1.png, 
> koji-kinoshita-tomasso-02.png, koji-kinoshita-tomasso.svg, 
> koji-kinoshita-tomasso-updated.png, koji-kinoshita-tomasso-updated.svg, 
> koji-kinoshita-tomasso-variations.png, koji-kinoshita-tomasso-variations.svg, 
> koji-kinoshita-tommaso.png, koji-kinoshita-tommaso.png, OpenNLP-koji-1.png, 
> OpenNLP-koji-1.png, OpenNLP-koji-2.png, OpenNLP.png, opennlp-variations.png, 
> page-example.png --fullpage.png, text4363.png
>
>
> The current logo was not changed for a long time (if ever). 
> Lets create a new fresh looking logo.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)