-
Notifications
You must be signed in to change notification settings - Fork 186
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
Deprecated the deep learning model #660
Comments
cc. @tomMoral too |
Hi @bruAristimunha ! I think on my side is the direction to go, since brain decode is well maintained. However I think can be useful to keep the current implementation in Tensorflow in order to ensure the full replicability of the MOABB benchmark. Maybe we can keep them in a separated folder. What do you think? |
Even if we remove it, the benchmark will be replicable by using the correct release. |
hey @carraraig, Scikit-keras is deprecated: adriangb/scikeras#334 And it is not working with the new version of scikit-learn. There is a complicated dependency issue, so we can not downgrade scikit-learn. Looks like the way to go is to use the scikit-learn API wrapper API that is back to Keras: Can you work on this? https://keras.io/api/utils/sklearn_wrappers/ If you can't, I think it's best to remove this moabb dependency and focus on braindecode. |
I'm holding moabb with sklearn < 1.16 for one week to give you time to answer. |
Hi @bruAristimunha ! I think that it is better to follow the direction of Braindecode, which is better maintained. |
okay @Hivenet-Igor! |
We have put a lot of effort into building a deep learning module, but clearly, this is not the best environment to maintain a deep learning library. I am proposing to deprecate the entire deep learning module and keep only the support for braindecode.
Currently, everything is already integrated into braindecode (EEGNeX, shallow, deep, eegnet and eegtcn), and it does not make much sense to have duplicate code.
Before officially removing the code duplication, a comparative study should be done and approved.
cc. @sylvchev and @carraraig
The text was updated successfully, but these errors were encountered: