----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27820/#review61348 -----------------------------------------------------------
datafu-pig/src/main/macros/nlp/tf_idf.pig <https://reviews.apache.org/r/27820/#comment102923> Shouldn't this be SUM? - Matthew Hayes On Nov. 10, 2014, 6:33 p.m., Russell Jurney wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27820/ > ----------------------------------------------------------- > > (Updated Nov. 10, 2014, 6:33 p.m.) > > > Review request for DataFu, pig, Joseph Adler, Jakob Homan, Matthew Hayes, and > Sam Shah. > > > Repository: datafu > > > Description > ------- > > DATAFU-61 - Add TF-IDF Macro to DataFu > > > Diffs > ----- > > datafu-pig/src/main/macros/nlp/tf_idf.pig PRE-CREATION > datafu-pig/src/test/macros/nlp/test_tf_idf.pig PRE-CREATION > > Diff: https://reviews.apache.org/r/27820/diff/ > > > Testing > ------- > > Works for me, but testing not automated. See > https://issues.apache.org/jira/browse/DATAFU-61 > > > Thanks, > > Russell Jurney > >
