biocore / Evident-initial-demo

Elucidating sampling effort for microbial analysis studies
7 stars 12 forks source link

One single sample selection #22

Closed antgonza closed 12 years ago

ElDeveloper commented 12 years ago

@antgonza and I briefly discussed about creating unit tests for evident.error but the following came along:

Traceback (most recent call last):
 File "svn_programs/evident/evident/evident/error.py", line 16, in <module>
   from mod_python import Session
 File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/mod_python/Session.py", line 21, in <module>
   import apache, Cookie
 File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/mod_python/apache.py", line 30, in <module>
   import _apache
ImportError: No module named _apache

Furthermore when trying to import the module from a python interactive shell the following happens:

>>> import evident.error
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "evident/error.py", line 16, in <module>
    from mod_python import Session
ImportError: No module named mod_python
>>> 

The solution is to either remove error.py or try and solve this. Maybe @douginator2000 may have some suggestions?

antgonza commented 12 years ago

I believe that the tests for error.py are important but not required at this time. I think that to move forward we need to verify that this code is fine, merge the pull request, and then create an issue with the improvement. Anybody else?

ElDeveloper commented 12 years ago

Makes perfect sense.

I will work on pull request #21 to solve conflicts that could arise by merging this pull request.