I think the req was for a few lines above on the creation of the given how
its populated. I don't care though, whatever minimizes the amount of time
to get this merged
On Mar 21, 2014 9:11 AM, "Jai Ram Rideout" <notifications@github.com> wrote:

> In qiime/util.py:
>
> > +                                     % (header, sample_id))
> > +                else:
> > +                    merged_data[sample_id][header] = value
> > +
> > +        # Now, convert what we have seen into a normal dict
> > +        normal_dict = {}
> > +        for sample_id in all_sample_ids:
> > +            if sample_id not in normal_dict:
> > +                normal_dict[sample_id] = {}
> > +
> > +            for header in all_headers:
> > +                normal_dict[sample_id][header] = \
> > +                    merged_data[sample_id][header]
> > +
> > +        # and create a MetadataMap object from it; concatenate comments
> > +        normal_dict = Dict2D(normal_dict)
>
> Cool, thanks. @wasade <https://github.com/wasade> was there a reason you
> wanted it to be a Dict2D?
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/biocore/qiime/pull/1302/files#r10839352>
> .
>