Yea, okay that makes sense, especially since it's been updated a bunch
previously without adding anything/anyone.


On Sat, May 10, 2014 at 12:25 PM, Luke Vilnis <notifications@github.com>wrote:

> In src/main/scala/cc/factorie/app/nlp/coref/CorefTrainer.scala:
>
> > @@ -0,0 +1,324 @@
> > +package cc.factorie.app.nlp.coref
> > +
> > +import cc.factorie.app.nlp.{DocumentAnnotatorPipeline, MutableDocumentAnnotatorMap, Document}
> > +import cc.factorie.app.nlp.wordnet.WordNet
> > +import cc.factorie.util.{Trackable, HyperparameterMain, TimingCollector, Trackers}
> > +import cc.factorie.app.nlp.phrase._
> > +import cc.factorie.app.nlp.load.LoadConll2011
> > +import cc.factorie.app.nlp.ner.{ConllChainNer, NerTag}
> > +
> > +
> > +/**
>
> I don't think we should have these attribution comments anywhere in
> Factorie, as that's what version control is for. But that might be a battle
> for another day. However, I think with a file like this that has seen a lot
> of different authors (including myself and David), we should have no line
> like this.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/factorie/factorie/pull/163/files#r12504902>
> .
>



-- 
Caitlin Cellier
University of Massachusetts, Amherst
Cellier@cs.umass.edu