-
Notifications
You must be signed in to change notification settings - Fork 51
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
Join efforts? #32
Comments
Hi @mpariente, I'm interested in DNN-based source separation. |
Great!
Well, there are architectures here that we don't have in Asteroid, it would be super cool to have them there. Have you had a look at our source code and the organization? |
I have not read the constraints, so I will have a look. I've confirmed that DANet separates the mixtures. But I have not checked the score (e.g. SDR improvement). I use a free Google Colaboratory plan to make sure my implementation works, so it may take some time, but I'd like to join efforts. |
There are also some Asteroid users that might be able to run your architectures on bigger hardware. Feel free to join Asteroid's slack workspace to start discussing 😉 |
Hi @tky823, nice repo !
We'd welcome most of this code in Asteroid if you'd like to contribute 😃 Would you?
Cheers,
The text was updated successfully, but these errors were encountered: