lxdv / ecg-classification

ECG Arrhythmia classification using CNN
MIT License
241 stars 71 forks source link

Windows incompatibility issue of label name "\" #24

Closed desmondlzy closed 3 years ago

desmondlzy commented 4 years ago

Hi, thanks a lot for your work!

I've come across some issues when trying to annotate the raw data on my windows machine. Taking a peek into your code, I believe the label name "\" is real problematic. I guess the root of the evil is the difference in the naming convention of file system between Windows and macOS/Linux... So when trying to create a file with "\" or "\" in its name, the system gives an error. (Actually os.path.join will give an erroneous result "/" on my machine).

FYI, I am using python 3.7.3 and windows 10 1903.

lxdv commented 3 years ago

Hello, @desmondlzy

Thank you for your feedback! Appreciate it very much! And SUPER SORRY for the late response.

I've used UNIX for the development that's why I didn't manage this behavior. If you still need this to be resolved, please, let me know.

Best regards, Alex

Echonamesli commented 2 years ago

subprocess.CalledProcessError: Command '['python3', 'dataset-generation.py', '--file', '../mit-bih/mit-bih-arrhythmia-database-1.0.0\100']' returned non-zero exit status 9009. 0%| | 0/49 [00:00<?, ?it/s] 我也遇到了上述问题 我用的是windows操作系统 请问下该怎么解决呢?很需要大家的帮助 万分感谢~~