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

Python2andpython3 #331

Closed alaindomissy closed 8 years ago

alaindomissy commented 8 years ago

Preliminary work for python2 and python3 compatibility. Not ready to merge. Still need to resolve unicode issues with datapackage save functions.

olgabot commented 8 years ago

Oh wow I'm looking again and I didn't realize that future works for python2 as well. In the past I've used the "six" module. Is that not necessary?

On Tue, Apr 26, 2016 at 11:53 AM Alain Domissy notifications@github.com wrote:

Preliminary work for python2 and python3 compatibility.

Still need to resolve unicode issues with datapackage save functions.

You can view, comment on, or merge this pull request online at:

https://github.com/YeoLab/flotilla/pull/331 Commit Summary

  • python and 3 compatibility, renamed test files, added tests
  • python2 and python3 compatibility

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub https://github.com/YeoLab/flotilla/pull/331

olgabot commented 8 years ago

While you're at it, can you replace the "print" statements with sys.stdout.write instead?

On Tue, Apr 26, 2016 at 11:55 AM Olga Botvinnik obotvinn@ucsd.edu wrote:

Oh wow I'm looking again and I didn't realize that future works for python2 as well. In the past I've used the "six" module. Is that not necessary?

On Tue, Apr 26, 2016 at 11:53 AM Alain Domissy notifications@github.com wrote:

Preliminary work for python2 and python3 compatibility.

Still need to resolve unicode issues with datapackage save functions.

You can view, comment on, or merge this pull request online at:

https://github.com/YeoLab/flotilla/pull/331 Commit Summary

  • python and 3 compatibility, renamed test files, added tests
  • python2 and python3 compatibility

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub https://github.com/YeoLab/flotilla/pull/331