Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Is there anyone convert it into py3.x version? #17

Open
Stevewsw opened this issue Jul 23, 2018 · 4 comments
Open

Is there anyone convert it into py3.x version? #17

Stevewsw opened this issue Jul 23, 2018 · 4 comments

Comments

@Stevewsw
Copy link

it is grateful if anyone rewrite it by py3.x .

@Zyjacya-In-love
Copy link

I have the same requirement. Have you solved it

@genaris
Copy link
Contributor

genaris commented Mar 28, 2020

Hi all, I'm working on a migration to Python 3 and Tensorflow 1.x. Seems to be working fine under Python 3.5.4 and tensorflow-gpu-1.15.2. Check my fork.

Anyway I'll send a pull request to the authors so they can test it and confirm it is fine (there were several changes to the RNN API since this original work has been done, and still some others to do if looking forward to a migration to tf 2.x).

@wangzhang021
Copy link

Hi all, I'm working on a migration to Python 3 and Tensorflow 1.x. Seems to be working fine under Python 3.5.4 and tensorflow-gpu-1.15.2. Check my fork.

Anyway I'll send a pull request to the authors so they can test it and confirm it is fine (there were several changes to the RNN API since this original work has been done, and still some others to do if looking forward to a migration to tf 2.x).

Hi, @genaris!
I am trying to reproduce this project, and get in trouble with the bugs.
When executing your work, I still can't succeed.
I failed in prepare_physionet.py:
_, _, ann = list(zip(*reader_ann.records()))

Returns
TypeError: catching classes that do not inherit from BaseException is not allowed

I reinstalled the dependency mne version to 0.15.2, but it does not work.
And my Python is 3.6
Could you please help me out? Thank you!

@genaris
Copy link
Contributor

genaris commented Apr 6, 2020

Hi @wangzhang021 , you are right. prepare_physionet.py was not updated (I've downloaded data with original py2.7 setup), so the repo was not in a consistent state.

I've just commited some changes: it should work now on python3 with mne 0.2.0 (latest version). However, it is still work in progress, as prepare_physionet.py code is using a deprecated parameter (scaling_time) that will not be available on mne 0.2.1. Hopefully I'll update it in the following days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants