search

LBHB / NEMS0

THIS VERSION OF NEMS IS NO LONGER SUPPORTED. PLEASE USE THE NEW NEMS REPOSITORY OR INSTALL NEMS_DB TO GET NEMS0 SUPPORT.
GNU General Public License v3.0
8 stars 4 forks source link

h5py should be included in the install_requires list #42

Closed neheller closed 6 years ago

neheller commented 6 years ago

Since h5py is imported and used in signal.py, it seems to be part of the minimal requirements for making the package run.

svdavid commented 6 years ago

@neheller Thanks for the feedback! I added the h5py dependency to the docs branch. We should be able to pull that over to master shortly.

bburan commented 6 years ago

Did you add it to setup.py?

On Mon, Apr 9, 2018 at 1:54 PM, Stephen D notifications@github.com wrote:

@neheller https://github.com/neheller Thanks for the feedback! I added the h5py dependency to the docs branch. We should be able to pull that over to master shortly.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/LBHB/NEMS/issues/42#issuecomment-379890895, or mute the thread https://github.com/notifications/unsubscribe-auth/AAT5Qj5GC_mhrjduXvnF91OcfjHMxdV7ks5tm8qQgaJpZM4TNIBl .