Adding audio-to-audio task.
#93
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Removed `audio-source-separation` (it's not `audio-to-audio` and it is a bit more general). Also the previous version was flawed as there was no way to return multiple channels. - Decision to return base64 blobs in JSON. It's not super shiny but it does the work and enabled easy ways to add annotations for audio streams (like the instrument labels for source separation). - Added GZipMiddleware within common, this help base64 encoded json be omewhat ok in terms of size. - Known caveat. Currently we only send 1 channel to the input of the pipeline. At the moment of this commit, no model requires stereo or multi channel input, so keeping that change for later. - community framework update will follow. - Because of dockerized versionning we can safely delete `audio-source-separation`. Old `api-inference-community` will continue to support those and the hub too. But if you want to use > 0.0.7 then you will need to migrate.
osanseviero
reviewed
Jun 9, 2021
api-inference-community/docker_images/common/tests/test_api_audio_to_audio.py
Show resolved
Hide resolved
osanseviero
approved these changes
Jun 9, 2021
|
This is awesome 🚀 Thank you! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
audio-source-separation(it's notaudio-to-audioand it is abit more general). Also the previous version was flawed as there was no way
to return multiple channels.
the work and enabled easy ways to add annotations for audio streams
(like the instrument labels for source separation).
omewhat ok in terms of size.
pipeline. At the moment of this commit, no model requires stereo or multi
channel input, so keeping that change for later.
audio-source-separation. Oldapi-inference-communitywill continue tosupport those and the hub too. But if you want to use > 0.0.7 then you will
need to migrate.