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

Kinect WSJ Recipe #131

Merged
merged 5 commits into from
May 31, 2020
Merged

Kinect WSJ Recipe #131

merged 5 commits into from
May 31, 2020

Conversation

sunits
Copy link
Collaborator

@sunits sunits commented May 30, 2020

  1. Contains recipe to train Deep Clustering on Kinect WSJ

TODO:
Multichannel Deep Clustering

sunits added 2 commits May 30, 2020 22:01
1. Contains recipe to train Deep Clustering on Kinect WSJ

TODO:
Include the multichannel  of DC
Copy link
Collaborator

@mpariente mpariente left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot !
Can you add a README.md directly into egs/kinect-wsj that has a small description of the dataset, a link to your repo and a way to cite the paper please?
Also you can add it to the main README in the supported dataset section.

There are few minor format issues that could be fixed with a linter (black for example)

egs/kinect-wsj/DeepClustering/local/preprocess_wsj0mix.py Outdated Show resolved Hide resolved
egs/kinect-wsj/DeepClustering/local/preprocess_wsj0mix.py Outdated Show resolved Hide resolved
egs/kinect-wsj/DeepClustering/run.sh Outdated Show resolved Hide resolved
os.path.join(exp_dir, 'checkpoints/final.pth'))


# TODO:Should ideally be inherited from wsj0-mix
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd love to reduce the copy pasting from recipes to recipes but apart from adding this things into the source code, I didn't find a good solution yet. Any good idea welcome !

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is Deep clustering the only recipe which requires a custom System class?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, also in DNS challenge and the two step recipe

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DC models can probably go into blocks.
System callbacks can handle common_steps I think. Will have to analyse a bit more to see if it gets complicated.

asteroid/data/__init__.py Outdated Show resolved Hide resolved
@mpariente
Copy link
Collaborator

Tests are failing because Zenodo's sandbox is down, no link to your PR.

@mpariente mpariente merged commit b172619 into master May 31, 2020
@mpariente
Copy link
Collaborator

Thanks that's perfect for now !
"maskter": muscle memory spotted 😉

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

Successfully merging this pull request may close these issues.

2 participants