YeoLab / flotilla

Reproducible machine learning analysis of gene expression and alternative splicing data
http://yeolab.github.io/flotilla/docs
BSD 3-Clause "New" or "Revised" License
121 stars 26 forks source link

python 3 compatibility #23

Open olgabot opened 10 years ago

olgabot commented 10 years ago

get rid of all print statements and replace with sys.stdout.write, e.g. https://github.com/YeoLab/flotilla/blob/master/flotilla/_frigate_compute.py#L855

gpratt commented 10 years ago

Have you looked into six? On Jun 16, 2014 1:36 PM, "Olga Botvinnik" notifications@github.com wrote:

get rid of all print statements and replace with sys.stdout.write, e.g. https://github.com/YeoLab/flotilla/blob/master/flotilla/_frigate_compute.py#L855

— Reply to this email directly or view it on GitHub https://github.com/YeoLab/flotilla/issues/23.

olgabot commented 10 years ago

yep, we'll use that, but first we gotta get rid of everything that doesn't use print as a function

olgabot commented 9 years ago

Labeling as "beginner-friendly" because it's well-defined in that if you use Python3, run all the tests and make sure they execute. Will need to use six for things like range (which in Python3 is the equivalent of xrange, and thus xrange is deprecated in Python 3)

olgabot commented 9 years ago

Goal: Python 3 compatibility by Scipy 2015 (July 6th, 2015)

olgabot commented 8 years ago

Wow great! Your help will be greatly appreciated! You have two options.

  1. There is a PR from Jessica Lettes who did a lot of work to update the code, so you can pull and rebase. This was last year And there may be more changes since
  2. Use the "six" python package to fix all tests that fail in python 3. Do "make test" in the flotilla directory to run the test suite

On Thu, Apr 14, 2016, 02:51 Sourav Singh notifications@github.com wrote:

I would like to work on this issue.How do I start?

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/YeoLab/flotilla/issues/23#issuecomment-209856136