Skip to content

Uniform kwargs for processors of audio-text models#32906

Draft
leloykun wants to merge 28 commits intohuggingface:mainfrom
leloykun:fc--uniformize-processor-kwargs-audio-text
Draft

Uniform kwargs for processors of audio-text models#32906
leloykun wants to merge 28 commits intohuggingface:mainfrom
leloykun:fc--uniformize-processor-kwargs-audio-text

Conversation

@leloykun
Copy link
Copy Markdown
Contributor

@leloykun leloykun commented Aug 21, 2024

What does this PR do?

TODO Models:

  • Clap
  • CLVP
  • MusicGen Melody
  • Qwen2 Audio
  • Seamless M4T
  • SpeechT5
  • Wav2Vec2 Bert

TODO tests

  • Add audio-text-specific processor tests
  • Remove unnecessary/duplicated tests

Models with special args (will not be done in this PR):

  • PopPiano

Models with weird _in_target_context_manager logic (will not be done in this PR):

  • MusicGen
  • SpeechToText
  • Wav2Vec2
  • Wav2Vec2 w/ LM
  • Whisper

Fixes # (issue)

Who can review?

@zucchini-nlp @molbap @yonigozlan

@leloykun leloykun mentioned this pull request Aug 21, 2024
40 tasks
@leloykun
Copy link
Copy Markdown
Contributor Author

@zucchini-nlp the tests are failing because of this: #32921

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.

1 participant