Github user hsaputra commented on a diff in the pull request:
https://github.com/apache/incubator-joshua/pull/15#discussion_r64144497
--- Diff: src/main/java/org/apache/joshua/decoder/JoshuaConfiguration.java
---
@@ -587,7 +588,7 @@ else if (fds[1].toLowerCase().equals("http"))
Github user thammegowda commented on the pull request:
https://github.com/apache/incubator-joshua/pull/15#issuecomment-220796532
@KellenSunderland Thanks for the review and feedback. I fixed them.
However, we are left with 200+ `System.out.print` and 200+
`System.err.print`
Dynamite Chris. Looks like we got it covered pretty well on our side :)
Please post anything here as well so that others can benefit. That goes
without saying :)
Have a nice weekend troops :)
On Fri, May 20, 2016 at 9:50 AM, <
dev-digest-h...@joshua.incubator.apache.org> wrote:
>
>
> From:
[
https://issues.apache.org/jira/browse/JOSHUA-264?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15294936#comment-15294936
]
ASF GitHub Bot commented on JOSHUA-264:
---
Github user KellenSunderland commented on the pull request:
Github user KellenSunderland commented on the pull request:
https://github.com/apache/incubator-joshua/pull/13#issuecomment-220772283
Hey Thamme, thanks for the great PR. Haven't checked the build yet myself
but the commit looks good to me.
---
If your project is set up for it, you
Github user KellenSunderland commented on a diff in the pull request:
https://github.com/apache/incubator-joshua/pull/15#discussion_r64132137
--- Diff: src/main/java/org/apache/joshua/util/CompareGrammars.java ---
@@ -134,15 +132,15 @@ public static void compareValues(File
Github user KellenSunderland commented on a diff in the pull request:
https://github.com/apache/incubator-joshua/pull/15#discussion_r64132143
--- Diff: src/main/java/org/apache/joshua/decoder/JoshuaDecoder.java ---
@@ -84,12 +79,13 @@ public static void main(String[] args) throws
Github user KellenSunderland commented on a diff in the pull request:
https://github.com/apache/incubator-joshua/pull/15#discussion_r64132090
--- Diff:
src/main/java/org/apache/joshua/util/encoding/FeatureTypeAnalyzer.java ---
@@ -122,7 +123,7 @@ public void inferTypes(boolean
Github user KellenSunderland commented on a diff in the pull request:
https://github.com/apache/incubator-joshua/pull/15#discussion_r64132048
--- Diff:
src/main/java/org/apache/joshua/util/encoding/FeatureTypeAnalyzer.java ---
@@ -122,7 +123,7 @@ public void inferTypes(boolean
Github user KellenSunderland commented on a diff in the pull request:
https://github.com/apache/incubator-joshua/pull/15#discussion_r64131989
--- Diff: src/main/java/org/apache/joshua/util/CompareGrammars.java ---
@@ -117,11 +115,11 @@ public static void compareValues(File
Github user KellenSunderland commented on a diff in the pull request:
https://github.com/apache/incubator-joshua/pull/15#discussion_r64131955
--- Diff: src/main/java/org/apache/joshua/util/Cache.java ---
@@ -104,20 +104,18 @@ public Cache() {
@Override
public V
Github user KellenSunderland commented on a diff in the pull request:
https://github.com/apache/incubator-joshua/pull/15#discussion_r64131960
--- Diff: src/main/java/org/apache/joshua/util/Cache.java ---
@@ -104,20 +104,18 @@ public Cache() {
@Override
public V
Github user KellenSunderland commented on a diff in the pull request:
https://github.com/apache/incubator-joshua/pull/15#discussion_r64131952
--- Diff: src/main/java/org/apache/joshua/util/Cache.java ---
@@ -104,20 +104,18 @@ public Cache() {
@Override
public V
Github user KellenSunderland commented on a diff in the pull request:
https://github.com/apache/incubator-joshua/pull/15#discussion_r64131932
--- Diff: src/main/java/org/apache/joshua/tools/GrammarPackerCli.java ---
@@ -146,7 +147,7 @@ public static void main(String[] args) throws
Github user KellenSunderland commented on a diff in the pull request:
https://github.com/apache/incubator-joshua/pull/15#discussion_r64131711
--- Diff:
src/main/java/org/apache/joshua/decoder/ff/lm/buildin_lm/TrieLM.java ---
@@ -141,7 +138,7 @@ public TrieLM(ArpaFile arpaFile)
Github user KellenSunderland commented on a diff in the pull request:
https://github.com/apache/incubator-joshua/pull/15#discussion_r64131682
--- Diff:
src/main/java/org/apache/joshua/decoder/ff/lm/bloomfilter_lm/BloomFilterLanguageModel.java
---
@@ -62,7 +63,7 @@
/**
Github user KellenSunderland commented on a diff in the pull request:
https://github.com/apache/incubator-joshua/pull/15#discussion_r64131583
--- Diff: src/main/java/org/apache/joshua/decoder/ff/lm/ArpaFile.java ---
@@ -180,21 +177,21 @@ public Vocabulary getVocab() {
Github user KellenSunderland commented on a diff in the pull request:
https://github.com/apache/incubator-joshua/pull/15#discussion_r64131585
--- Diff: src/main/java/org/apache/joshua/decoder/ff/lm/ArpaFile.java ---
@@ -209,10 +206,10 @@ public int getOrder() throws
Github user KellenSunderland commented on a diff in the pull request:
https://github.com/apache/incubator-joshua/pull/15#discussion_r64131303
--- Diff: src/main/java/org/apache/joshua/corpus/Vocabulary.java ---
@@ -45,6 +47,7 @@
public class Vocabulary {
+
Github user KellenSunderland commented on a diff in the pull request:
https://github.com/apache/incubator-joshua/pull/15#discussion_r64131297
--- Diff: src/main/java/org/apache/joshua/decoder/Decoder.java ---
@@ -786,7 +783,7 @@ private void initializeTranslationGrammars() throws
20 matches
Mail list logo