sounds reasonable


On Thu, Aug 15, 2013 at 3:24 PM, Jai Ram Rideout
<notifications@github.com>wrote:

> In python-code/biom/commands/summarize_biom_table.py:
>
> > +__credits__ = ["Greg Caporaso"]
> > +__license__ = "GPL"
> > +__version__ = "1.1.2-dev"
> > +__maintainer__ = "Greg Caporaso"
> > +__email__ = "gregcaporaso@gmail.com"
> > +
> > +class SummarizeBiomTable(Command):
> > +    """
> > +     Example usage:
> > +      from biom.commands.summarize_biom_
> table import SummarizeBiomTable
> > +      from biom.parse import parse_biom_table
> > +      c = SummarizeBiomTable()
> > +      t = parse_biom_table(open("table.biom"))
> > +      result = c(table=t)
> > +      result = c(table=t,qualitative=True)
> > +      result = c(table=t,qualitative=True,input_fp="table.biom")
>
> Open the pull request and click 'Show outdated diff' to see the discussion.
>
> No, MD5 would only be computed within Command.run if the second element
> in the tuple isn't None (i.e., it has a file-like object) and
> suppress_md5_sum isn't True. The input handler would only be returning a
> tuple containing the parsed table and the filehandle.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/biom-format/biom-format/pull/161/files#r5801812>
> .
>