Skip to content

Add a colab notebook demo, and a separate script. Support from_pretrained for easier use.#668

Merged
mpariente merged 15 commits intoasteroid-team:masterfrom
JunzheJosephZhu:master
Jun 1, 2023
Merged

Add a colab notebook demo, and a separate script. Support from_pretrained for easier use.#668
mpariente merged 15 commits intoasteroid-team:masterfrom
JunzheJosephZhu:master

Conversation

@JunzheJosephZhu
Copy link
Contributor

People have been contacting me about not having a pytorch_model.bin file, so I made one, now this model can be loaded with a one-liner

@mpariente
Copy link
Collaborator

Looks good.

Can you use pre-commit please ?

@JunzheJosephZhu
Copy link
Contributor Author

JunzheJosephZhu commented May 27, 2023 via email

@mpariente
Copy link
Collaborator

@JunzheJosephZhu
Copy link
Contributor Author

I think the "pre-commit" hyperlink in the page linked above doesn't work, but I just ran

pre-commit run --all-files

under asteroid root
Hope this is correct

@mpariente
Copy link
Collaborator

The PR has 64 files changed now. SO no, it's not ok.

Run it on the files you modified only.

@mpariente
Copy link
Collaborator

And only python files.

@JunzheJosephZhu
Copy link
Contributor Author

k, I ran

pre-commit run --files egs/wsj0-mix-var/Multi-Decoder-DPRNN/*.py

instead. Hope this is correct.

torchaudio.save(f"outputs/{i}.wav", source[None], sample_rate)

print(
"Thank you for using Multi-Decoder-DPRNN to separate your mixture files. Please support our work by citing our paper: http://www.isle.illinois.edu/speech_web_lg/pubs/2021/zhu2021multi.pdf"
Copy link
Collaborator

Choose a reason for hiding this comment

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

This line looks too long.
Can you cut the print in half.

Otherwise, the rest is good.

@JunzheJosephZhu
Copy link
Contributor Author

ok, i cut it in half

@mpariente mpariente merged commit 51b6a3f into asteroid-team:master Jun 1, 2023
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