search

shbhuk / barycorrpy

Python version of Barycorr
GNU General Public License v3.0
37 stars 6 forks source link

setup.py needs to have barycorrpy.tests #29

Closed indiajoe closed 4 years ago

indiajoe commented 4 years ago

Currently the module barrycorrpy.tests is missing in the setup.py file. This has to be added to the packages list.

packages=['barycorrpy','barycorrpy.tests'],

Otherwise, pip will not install the sub module while package is installed directly. This will result in a ModuleNotFoundError: No module named 'barycorrpy.tests' error while trying to import barrycorpy.

shbhuk commented 4 years ago

Hi Joe, might that be because the latest version on pip is 0.2.2.1, whereas the unit tests were added in 0.2.3?

indiajoe commented 4 years ago

Yes, we ran into this problem when we tried to install the latest version from github directly. The version on PyPi might be fine.

shbhuk commented 4 years ago

Try now.. https://github.com/shbhuk/barycorrpy/commit/eed5e3604220e31b3e94b70e046a2c7b7a26e710