Build failed in Jenkins: joshua_master #69

See Changes: [post] javadoc fixes -- [...truncated 4910 lines...] 72/286 KB 76/286 KB 80/286 KB 84/286 KB 88/286 KB 92/286 KB 96/286 KB 100/286 KB 104/286 KB 108/286 KB

Build failed in Jenkins: joshua_master #68

See Changes: [antonio.t] Adding the CHRF metric for tuning [antonio.t] Adding the CHRF metric for tuning [post] don't return formatted string; updated test case [post] Reverted commit 6d2213a20b74432fc7cb131c732f7507b74053e9, removed

[GitHub] incubator-joshua issue #26: Owner Ids are maintained in separate mapping now

Github user mjpost commented on the issue: https://github.com/apache/incubator-joshua/pull/26 Great, this checks out for me. Since it contains the class LM fix, too, though, I'll wait to merge until those tests pass. --- If your project is set up for it, you can reply to this email

[GitHub] incubator-joshua issue #25: ClassLMs: fixed a bug with class-based lms not m...

Github user mjpost commented on the issue: https://github.com/apache/incubator-joshua/pull/25 This passes my regression tests. I am now testing it on an actual tuning / testing run for en-fi. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-joshua issue #24: Maven multi-module project layout proposal

Github user logogin commented on the issue: https://github.com/apache/incubator-joshua/pull/24 Hi @mjpost . I wanted general opinion on the proposal but looks mostly people support it. Some things to take into account if that will go into master: * joshua-servet contains only

[GitHub] incubator-joshua issue #24: Maven multi-module project layout proposal

Github user mjpost commented on the issue: https://github.com/apache/incubator-joshua/pull/24 Thanks, @logogin. I have only glanced at this, but it seems like a good idea. I hope to be able to take a look tonight, and expect all will be well. Do you want more discussion, or should I

[GitHub] incubator-joshua issue #24: Maven multi-module project layout proposal

Github user logogin commented on the issue: https://github.com/apache/incubator-joshua/pull/24 Re-based on top of master. I also split the change into two separate commits to simplify the review: the 1st one is just moving files, no changes; the 2nd contains actual modifications

[GitHub] incubator-joshua pull request #27: Adding the CHRF metric for tuning

GitHub user antot opened a pull request: https://github.com/apache/incubator-joshua/pull/27 Adding the CHRF metric for tuning This pull request adds the CHRF metric for tuning. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-joshua pull request #26: Owner Ids are maintained in separate mapp...

GitHub user fhieber opened a pull request: https://github.com/apache/incubator-joshua/pull/26 Owner Ids are maintained in separate mapping now Removed owner ids from Vocabulary. These are now maintained in their own mapping. Fixes a bug with multiple packed grammars that would

[GitHub] incubator-joshua pull request #25: ClassLMs: fixed a bug with class-based lm...

GitHub user fhieber opened a pull request: https://github.com/apache/incubator-joshua/pull/25 ClassLMs: fixed a bug with class-based lms not mapping to class ids for estimateCost() Also refactored the code a little bit to have StateMinimizingLanguageModels support classes as well.