hovren / crisp

Camera-to-IMU calibration and synchronization toolbox
GNU General Public License v3.0
206 stars 74 forks source link

Please add Cython to the list of dependencies. #10

Closed Jigyasu closed 6 years ago

hovren commented 7 years ago

To which list should it be added?

In the source packages/releases (i.e. on PyPI) the Cython-generated C-files are included, and thus Cython is not needed for installation. Therefore you only need Cython if you are cloning the repository from Github. So, adding Cython to the requires= or install_requires= options in setup() is going to pull in Cython for everyone even though they don't need it, and I am hesistant to do so.