mattjj / pyhsmm

MIT License
546 stars 173 forks source link

reorganizing pyhsmm into standard Python package format #33

Closed yarden closed 9 years ago

yarden commented 9 years ago

After you look it over, please register the package with pypi. That way others can include pyhsmm in the requirements of their own setup.py packages and have it be installed automatically for users, without going through github.

Thanks! Best, Yarden

mattjj commented 9 years ago

This is tremendous, thank you!

(Let me know if you notice any mistakes working with pypi and all that.)

yarden commented 9 years ago

There's a minor pypi distribution bug that I missed and just fixed in pyhsmm and pybasicbayes repositories - sent you two pull requests. The ideal solution to this stuff is to have a conditional setup.py that checks for Python, and then, as you noted in one of the other issues, include the Cython-generated *.c only in the source distribution (without requiring pyx compilation). Theoretically only developers need to have Cython and compile pyx files.