dfm/emcee

Is numpy required for SETUP_REQUIRES

BastianZim opened this issue · 4 comments

Hi,

currrently, SETUP_REQUIRES lists numpy as specified here:

emcee/setup.py

Line 26 in bee076f

SETUP_REQUIRES = INSTALL_REQUIRES + [

I don't see any place where this is actually used during building though so I was wondering if that could be removed.

Thanks!

dfm commented

Good point - I have some vague recollection that that was added for a reason, but it definitely seems unnecessary!

Happy with me putting in a PR?

dfm commented

Please do!

PR is in #427