search

kylejusticemagnuson / pyti

Python library of various financial technical indicators
MIT License
646 stars 169 forks source link

Fixed Python 3 bug #5

Closed BernhardSchlegel closed 6 years ago

kylejusticemagnuson commented 6 years ago

Hi @BernhardSchlegel

Thanks for submitting a fix. I'll take a look when I have time within the next day.

So far I have put no effort into making sure this library works with Python 3. It's been on my TODO list for a while so I might take a look into the effort that would take this weekend.

BernhardSchlegel commented 6 years ago

Thanks for the reply. Turns out there are more issues with Python 3. Literally every time you used a map()... Don't know what the best way would be to cope with this. Wrapping in a list (list(map())) is my current workaround...

kylejusticemagnuson commented 6 years ago

@BernhardSchlegel I have updated pyti so it should work with both Python 2 and 3 now. Let me know if you have any issues