A better resolution to a possible circular dependency.
authorDiane Trout <diane@caltech.edu>
Wed, 9 Jan 2013 00:11:13 +0000 (16:11 -0800)
committerDiane Trout <diane@caltech.edu>
Wed, 9 Jan 2013 00:11:13 +0000 (16:11 -0800)
commitb43a6b4b7b9649ec6cc433b9c38433e86fea096d
treed3f677e4f24549d6d0acc9e2daf5313c975a6a79
parent6824b9a5bd9eaeb092caf9b9fcb8afe1f4cdda96
A better resolution to a possible circular dependency.

The runfolder subdirectory processing tools (firecrest, bustard, gerald, etc.)
were importing runfolder for some common constants, however runfolder
imported them to actually build the runfolder structure.

My previous solution was to only include the imports for the
sub-directory processing in the function that used them.
However that lead to needing nested functions which seemed confusing.

What I did was move the common constants into pipelines.__init__
and just imported them from there.
23 files changed:
htsworkflow/pipelines/__init__.py
htsworkflow/pipelines/bustard.py
htsworkflow/pipelines/eland.py
htsworkflow/pipelines/firecrest.py
htsworkflow/pipelines/genomemap.py
htsworkflow/pipelines/gerald.py
htsworkflow/pipelines/ipar.py
htsworkflow/pipelines/retrieve_config.py
htsworkflow/pipelines/runfolder.py
htsworkflow/pipelines/summary.py
htsworkflow/pipelines/test/test_extract_results.py
htsworkflow/pipelines/test/test_genomemap.py
htsworkflow/pipelines/test/test_runfolder026.py
htsworkflow/pipelines/test/test_runfolder030.py
htsworkflow/pipelines/test/test_runfolder110.py
htsworkflow/pipelines/test/test_runfolder_casava_1_7.py
htsworkflow/pipelines/test/test_runfolder_ipar100.py
htsworkflow/pipelines/test/test_runfolder_ipar130.py
htsworkflow/pipelines/test/test_runfolder_pair.py
htsworkflow/pipelines/test/test_runfolder_rta.py
htsworkflow/pipelines/test/test_runfolder_rta160.py
htsworkflow/pipelines/test/test_runfolder_rta180.py
htsworkflow/pipelines/test/test_runfolder_rta1_12.py