reasonable, i will move that.


On Tue, Dec 10, 2013 at 11:21 AM, Jai Ram Rideout
<notifications@github.com>wrote:

> In tests/test_otu_significance.py:
>
> > +    run_grouped_correlation, CORRELATION_TEST_CHOICES,
> > +    grouped_correlation_formatter, correlation_row_generator,
> > +    run_correlation_test)
> > +from qiime.pycogent_backports.test import (assign_correlation_pval, fisher,
> > +    fisher_population_correlation)
> > +from numpy import array, hstack, corrcoef
> > +from numpy.random import seed
> > +from cogent.util.dict2d import Dict2D
> > +from qiime.util import get_tmp_filename
> > +from os import remove
> > +from qiime.parse import parse_mapping_file_to_dict, parse_otu_table
> > +from biom.parse import parse_biom_table
> > +from qiime.format import format_biom_table
> > +
> > +# globals used by certain tests.
> > +BT_IN_1 = '{"id": "None","format": "Biological Observation Matrix 1.0.0","format_url": "http://biom-format.org","type": "OTU table","generated_by": "testCode","date": "2013-08-20T15:48:21.166180","matrix_type": "sparse","matrix_element_type": "float","shape": [6, 6],"data": [[0,0,28.0],[0,1,52.0],[0,2,51.0],[0,3,78.0],[0,4,16.0],[0,5,77.0],[1,0,25.0],[1,1,14.0],[1,2,11.0],[1,3,32.0],[1,4,48.0],[1,5,63.0],[2,0,31.0],[2,1,2.0],[2,2,15.0],[2,3,69.0],[2,4,64.0],[2,5,27.0],[3,0,36.0],[3,1,68.0],[3,2,70.0],[3,3,65.0],[3,4,33.0],[3,5,62.0],[4,0,16.0],[4,1,41.0],[4,2,59.0],[4,3,40.0],[4,4,15.0],[4,5,3.0],[5,0,32.0],[5,1,8.0],[5,2,54.0],[5,3,98.0],[5,4,29.0],[5,5,50.0]],"rows": [{"id": "OTU1", "metadata": {"taxonomy": ["k__One"]}},{"id":
>   "OTU2", "metadata": {"taxonomy": ["k__Two"]}},{"id": "OTU3", "metadata": {"taxonomy": ["k__Three"]}},{"id": "OTU4", "metadata": {"taxonomy": ["k__Four"]}},{"id": "OTU5", "metadata": {"taxonomy": ["k__Five"]}},{"id": "OTU6", "metadata": {"taxonomy": ["k__Six"]}}],"columns": [{"id": "Sample1", "metadata": null},{"id": "Sample2", "metadata": null},{"id": "Sample3", "metadata": null},{"id": "Sample4", "metadata": null},{"id": "Sample5", "metadata": null},{"id": "Sample6", "metadata": null}]}'
>
> We usually put global test data at the bottom of the file. While certainly
> not part of our coding guidelines, I recommend moving these variables to
> maintain consistency (plus I want to power-trip just a little bit ;)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/qiime/qiime/pull/1091/files#r8241132>
> .
>