Fixed compatibility issues with Python 3

jma127 / pyltr

Python learning to rank (LTR) toolkit

BSD 3-Clause "New" or "Revised" License

463 stars 107 forks source link

Fixed compatibility issues with Python 3 #2

Closed soldni closed 7 years ago

soldni commented 7 years ago

Hello!

First of all, thanks for this package. It has saved me quite some time :smile:

I've made a few changes (fixing imports and range/xrange issues) to make the project compatible with Python 3. In my tests, it runs with no problem under both 2 and 3.

I've also fixed some typos in the example in the read me file.

Best, Luca

jma127 commented 7 years ago

Hi Luca, thanks so much for this contribution! I've been meaning to add python3 support for a while but never found the time as of yet. I'll test tomorrow morning and get it merged if everything checks out.

jma127 commented 7 years ago

Everything checks out & merged. Thanks again!

soldni commented 7 years ago

Sure, no problem! Glad I could help 😉