Skip to content

Conversation

samanklesaria
Copy link
Collaborator

@samanklesaria samanklesaria commented Aug 20, 2025

Now that all functions we deprecated in 2.8 have been removed, there is no need for the dropping_support family of decorators. This PR removes them. It also un-deprecates the stable ABI ported functions. Note that this PR depends no_wrap_ctc_decoder.

Copy link

pytorch-bot bot commented Aug 20, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/audio/4056

Note: Links to docs will display an error until the docs builds have been completed.

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@meta-cla meta-cla bot added the CLA Signed label Aug 20, 2025
@samanklesaria samanklesaria changed the base branch from main to pseudo_main August 20, 2025 17:35
@samanklesaria samanklesaria marked this pull request as ready for review August 20, 2025 18:30
@samanklesaria samanklesaria requested a review from a team as a code owner August 20, 2025 18:30
@pearu
Copy link
Collaborator

pearu commented Sep 2, 2025

This PR requires a rebase against the main branch.

@pearu pearu added this to the 2.9 milestone Sep 2, 2025
@samanklesaria samanklesaria changed the base branch from pseudo_main to main September 4, 2025 14:38
@samanklesaria samanklesaria changed the base branch from main to no_wrap_ctc_decoder September 4, 2025 14:38
@samanklesaria samanklesaria changed the base branch from no_wrap_ctc_decoder to pseudo_main September 4, 2025 14:39
@samanklesaria samanklesaria force-pushed the remove_dropping_support branch from c1e3840 to 299376b Compare September 4, 2025 14:42
@samanklesaria samanklesaria changed the base branch from pseudo_main to no_wrap_ctc_decoder September 4, 2025 14:47
@samanklesaria
Copy link
Collaborator Author

Sucessfully rebased against main. We just need to wait for #4055 to be merged.

@pearu pearu modified the milestones: 2.9, 2.10 Sep 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants