Closed nateanl closed 2 years ago
This is most likely not about this PR, but it is not immediately clear where the entry point is. This makes it very difficult to understand the train logic from user perspective.
I think this might exhibit the same painpoints of existing training framework where source of training config value is not clear.
How can I bring my own model if I want to replace the model with my own (with the same interface)?
@nateanl has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
@nateanl has updated the pull request. You must reimport the pull request before landing.
@nateanl has updated the pull request. You must reimport the pull request before landing.
@nateanl has updated the pull request. You must reimport the pull request before landing.
@nateanl has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
@nateanl has updated the pull request. You must reimport the pull request before landing.
@nateanl has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
The PR includes a training recipe for source separation that trains Conv-TasNet model on Libri2Mix dataset.