Skip to content

Conversation

rev2607
Copy link
Contributor

@rev2607 rev2607 commented Jul 22, 2025

What does this PR do?

Fixes a typo in the method name apply_transcrition_requestapply_transcription_request inside src/transformers/models/voxtral/processing_voxtral.py.

This PR addresses issue #39530. The method was misspelled and is now corrected for clarity and consistency. This change may also require updates in model cards if this method is referenced there.

Fixes #39530

Before submitting

  • This PR fixes a typo or improves the docs.
  • I have read the contributor guidelines.
  • This was discussed/approved via issue #39530.
  • No new documentation or tests were required for this minor change.

Who can review?

@speech models: @eustlb

@rev2607
Copy link
Contributor Author

rev2607 commented Jul 22, 2025

Hi @eustlb, this PR fixes the typo in the method name mentioned in #39530. Kindly review when you get a chance 😊

@rev2607 rev2607 force-pushed the fix-transcription-typo branch from 63f00ea to 0b6229d Compare July 22, 2025 08:27
Copy link
Member

@qubvel qubvel left a comment

Choose a reason for hiding this comment

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

Thanks for the fix! I also found this typo while looking into the demo code

https://huggingface.co/spaces/MohamedRashad/Voxtral/blob/main/app.py#L62

To make smooth fix we should also update

  • voxtral.md
  • fix typo in model cards on the Hub (3B and 24B)
  • make sure demos will work after the update (make corresponding PR's to fix the typo)

As an intermediate step, we can support both methods for a while, e.g.

# method with typo
def apply_transcrition_request(self, *args, **kwargs):
    warning.warn("Method is deprecated, please use `apply_transcription_request` ...")
    return apply_transcription_request(*args, **kwargs)

@eustlb
Copy link
Contributor

eustlb commented Jul 22, 2025

Thanks for finding this inconvenient typo... agree with @qubvel. @rev2607 can you add the suggested change so that we ensure we support both?
Once merged, I'll take care of updating the hub model cards.

@rev2607 rev2607 force-pushed the fix-transcription-typo branch from 5a52ef1 to c98c0ad Compare July 22, 2025 11:42
@rev2607
Copy link
Contributor Author

rev2607 commented Jul 22, 2025

Thanks for the feedback @qubvel and @eustlb! I've pushed the requested changes to fix the typo and add the temporary wrapper. Let me know what you think.

@qubvel
Copy link
Member

qubvel commented Jul 22, 2025

@rev2607 can you please update voxtral.md file in the repo as well?

@rev2607
Copy link
Contributor Author

rev2607 commented Jul 22, 2025

Thanks, @qubvel. I've updated voxtral.md as requested.

Copy link
Member

@qubvel qubvel left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@qubvel qubvel requested a review from eustlb July 22, 2025 15:51
@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@ebezzam ebezzam added the Audio label Jul 23, 2025
@qubvel
Copy link
Member

qubvel commented Jul 23, 2025

run-slow: voxtral

Copy link
Contributor

This comment contains run-slow, running the specified jobs:

models: ['models/voxtral']
quantizations: [] ...

@eustlb
Copy link
Contributor

eustlb commented Jul 23, 2025

Checking slow tests (likely compute related)
Edit: merging #39605 quickly to update to runners values

@eustlb
Copy link
Contributor

eustlb commented Jul 24, 2025

run-slow: voxtral

Copy link
Contributor

This comment contains run-slow, running the specified jobs:

models: ['models/voxtral']
quantizations: [] ...

@eustlb eustlb enabled auto-merge (squash) July 25, 2025 09:39
@eustlb eustlb disabled auto-merge July 25, 2025 09:54
@eustlb eustlb enabled auto-merge (squash) July 25, 2025 10:37
Copy link
Contributor

[For maintainers] Suggested jobs to run (before merge)

run-slow: voxtral

@eustlb eustlb merged commit 3b3f9c0 into huggingface:main Jul 25, 2025
19 checks passed
zaristei pushed a commit to zaristei/transformers that referenced this pull request Sep 9, 2025
…e#39572)

* fix(voxtral): correct typo in apply_transcription_request

* temporary wrapper: apply_transcrition_request

* Update processing_voxtral.py

* style: sort imports in processing_voxtral.py

* docs(voxtral): fix typo in voxtral.md

* make style

* doc update

---------

Co-authored-by: eustlb <[email protected]>
Co-authored-by: Eustache Le Bihan <[email protected]>
zaristei pushed a commit to zaristei/transformers that referenced this pull request Sep 9, 2025
…e#39572)

* fix(voxtral): correct typo in apply_transcription_request

* temporary wrapper: apply_transcrition_request

* Update processing_voxtral.py

* style: sort imports in processing_voxtral.py

* docs(voxtral): fix typo in voxtral.md

* make style

* doc update

---------

Co-authored-by: eustlb <[email protected]>
Co-authored-by: Eustache Le Bihan <[email protected]>
zaristei pushed a commit to zaristei/transformers that referenced this pull request Sep 9, 2025
…e#39572)

* fix(voxtral): correct typo in apply_transcription_request

* temporary wrapper: apply_transcrition_request

* Update processing_voxtral.py

* style: sort imports in processing_voxtral.py

* docs(voxtral): fix typo in voxtral.md

* make style

* doc update

---------

Co-authored-by: eustlb <[email protected]>
Co-authored-by: Eustache Le Bihan <[email protected]>
zaristei pushed a commit to zaristei/transformers that referenced this pull request Sep 9, 2025
…e#39572)

* fix(voxtral): correct typo in apply_transcription_request

* temporary wrapper: apply_transcrition_request

* Update processing_voxtral.py

* style: sort imports in processing_voxtral.py

* docs(voxtral): fix typo in voxtral.md

* make style

* doc update

---------

Co-authored-by: eustlb <[email protected]>
Co-authored-by: Eustache Le Bihan <[email protected]>
zaristei pushed a commit to zaristei/transformers that referenced this pull request Sep 9, 2025
…e#39572)

* fix(voxtral): correct typo in apply_transcription_request

* temporary wrapper: apply_transcrition_request

* Update processing_voxtral.py

* style: sort imports in processing_voxtral.py

* docs(voxtral): fix typo in voxtral.md

* make style

* doc update

---------

Co-authored-by: eustlb <[email protected]>
Co-authored-by: Eustache Le Bihan <[email protected]>
zaristei pushed a commit to zaristei/transformers that referenced this pull request Sep 9, 2025
…e#39572)

* fix(voxtral): correct typo in apply_transcription_request

* temporary wrapper: apply_transcrition_request

* Update processing_voxtral.py

* style: sort imports in processing_voxtral.py

* docs(voxtral): fix typo in voxtral.md

* make style

* doc update

---------

Co-authored-by: eustlb <[email protected]>
Co-authored-by: Eustache Le Bihan <[email protected]>
zaristei pushed a commit to zaristei/transformers that referenced this pull request Sep 9, 2025
…e#39572)

* fix(voxtral): correct typo in apply_transcription_request

* temporary wrapper: apply_transcrition_request

* Update processing_voxtral.py

* style: sort imports in processing_voxtral.py

* docs(voxtral): fix typo in voxtral.md

* make style

* doc update

---------

Co-authored-by: eustlb <[email protected]>
Co-authored-by: Eustache Le Bihan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Typo in apply_transcrition_request method name
6 participants