niekverw / Deep-Learning-Based-ECG-Annotator

Annotation of ECG signals using deep learning, tensorflow’ Keras
146 stars 49 forks source link

wfdb 2.0+ causes issues. #3

Open BrettMontague opened 6 years ago

BrettMontague commented 6 years ago

First of all, thanks for this. Secondly, wfdb 2.0+ breaks some of your code here, there are some different arguments, and when I fixed those up rdann was returning a tuple, not an object, in the new package.

I stopped there and just rolled back to wfdb 1.3.4 and it worked perfectly. You might want to make note of that in the readme or something.

niekverw commented 6 years ago

Thanks! yes I should list the package versions, I will at least add this.

niekverw commented 6 years ago

(You are also free to push any useful changes).

BehnamTaki commented 6 years ago

thank you @niekverw for this code. how to install specific version of wfdb package? that was impossible for me.

BrettMontague commented 6 years ago

It depends on your particular python environment @BehnamTaki, but in general you should be able to:

pip uninstall wfdb

to uninstall your current version of wfdb, and then

pip install wfdb==1.3.4

to install the 1.3.4 version of wfdb that you require.

If you happen to be using Anaconda:

conda install -c conda-forge wfdb=1.3.5 will probably work, or you can just use pip in Conda.

BehnamTaki commented 6 years ago

i appreciate you for putting time to answer my question