search

rachelss / SISRS

Site Identification from Short Read Sequences.
24 stars 15 forks source link

Python port #58

Closed anderspitman closed 6 years ago

anderspitman commented 6 years ago

@BobLiterman this PR represents a good jumping off point for moving everything over to Python. I finally got all the Python 3 stuff working.

In order to generate test data that I could use to verify the changes here, I needed to make a few modifications to bash SISRS. Those are represented in #57. That PR does not strictly need to be merged, but I wanted to make sure it was obvious what needed to change in order to compare a bash SISRS + Python 2 run against a sisrs-python + Python 3 run (this PR).

rachelss commented 6 years ago

@anderspitman could you submit this pull request to the python port branch. Let's leave master with bash for now.