analysiscenter / cardio

CardIO is a library for data science research of heart signals
https://analysiscenter.github.io/cardio/
Apache License 2.0
247 stars 78 forks source link

AttributeError: module 'cardio.dataset' has no attribute 'Batch' #2

Closed simonw closed 6 years ago

simonw commented 6 years ago

I got this while trying to follow the tutorial:

import cardio.dataset as ds
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-2-46c2fbe44381> in <module>()
----> 1 import cardio.dataset as ds

~/Dropbox/Development/cardio/cardio/__init__.py in <module>()
      2 import sys
      3 
----> 4 from .batch import *  # pylint: disable=wildcard-import
      5 from . import dataset  # pylint: disable=wildcard-import
      6 

~/Dropbox/Development/cardio/cardio/batch/__init__.py in <module>()
      1 """ ECG Batch """
----> 2 from .ecg_batch import EcgBatch
      3 from .ecg_dataset import EcgDataset

~/Dropbox/Development/cardio/cardio/batch/ecg_batch.py in <module>()
     16 
     17 
---> 18 class EcgBatch(ds.Batch):  # pylint: disable=too-many-public-methods,too-many-instance-attributes
     19     """Batch class for ECG signals storing.
     20 

AttributeError: module 'cardio.dataset' has no attribute 'Batch'

Here are the install steps I used - I had to manually install numba after getting an error about it being missing:

cd cardio
virtualenv --python=python3 venv
source venv/bin/activate
pip install .
pip install jupyter
jupyter notebook --port=8089
# I got ImportError: No module named 'numba'
pip install numba

I'm running Python 3.5 on OS X.

roman-kh commented 6 years ago

Thank you for letting us know.

Did you use the flag --recursive when you cloned the repo?

git clone --recursive https://github.com/analysiscenter/cardio.git

simonw commented 6 years ago

I did not! Will try that later tonight.

simonw commented 6 years ago

Thanks, that fixed the problem I was having. Everything works as expected now.