Perfectly fine by me!

Frank Austin Nothaft
fnothaft@alumni.stanford.edu
202-340-0466

On Oct 25, 2013, at 9:05 AM, Matt Massie <notifications@github.com> wrote:

> In adam-commands/src/main/scala/edu/berkeley/cs/amplab/adam/commands/PileupAggregator.scala:
> 
> > + */
> > +
> > +import edu.berkeley.cs.amplab.adam.util.{Args4jBase, Args4j}
> > +import net.sf.samtools.TextCigarCodec
> > +import edu.berkeley.cs.amplab.adam.avro.{ADAMRecord, ADAMPileup, Base}
> > +import org.kohsuke.args4j.Argument
> > +import spark.{RDD, SparkContext}
> > +import spark.SparkContext._
> > +import org.apache.hadoop.mapreduce.Job
> > +import parquet.hadoop.{ParquetOutputFormat, ParquetInputFormat}
> > +import parquet.avro.AvroReadSupport
> > +import edu.berkeley.cs.amplab.adam.predicates.LocusPredicate
> > +import parquet.hadoop.util.ContextUtil
> > +
> > +object PileupAggregator extends AdamCommandCompanion {
> > +  val commandName: String = "pileupAggregator"
> Is it ok if I change the name of the command to "aggregate_pileup"? I'd like the commands to be action words instead of nouns. I just change this myself before I push this change but I wanted to ask you first. Of course, I'm happy to take alternatives.
> 
> —
> Reply to this email directly or view it on GitHub.
>