search

ImSoErgodic / py-upset

A pure-python implementation of the UpSet suite of visualisation methods by Lex, Gehlenborg et al.
313 stars 57 forks source link

Python 2 support #2

Open agitter opened 8 years ago

agitter commented 8 years ago

It looks like only Python 3 is currently supported due to the way the optional arguments are provided to the plot function. Is that correct?

afrendeiro commented 8 years ago

True. Your fork works fine though. Thanks!

agitter commented 8 years ago

Thanks for commenting. I didn't yet examine the consequences of re-ordering the arguments in my fork. Does any other code make use of these optional arguments and need to be changed to be consistent?

ImSoErgodic commented 8 years ago

Forgive my delay, for some reason I wasn't receiving notifications for this repo.

The package was developed mainly with python 3 in mind, that's correct.

I will take a look at the changes and get back to you asap.