Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added an "s" to match the function name #7394

Merged
merged 1 commit into from
Mar 7, 2023
Merged

Conversation

camilodlt
Copy link
Contributor

Update the _BETA_TRANSFORMS_WARNING string that currently prints a function that does not exist. An "s" is added. From torchvision.disable_beta_transform_warning() to torchvision.disable_beta_transforms_warning()

Update the _BETA_TRANSFORMS_WARNING string that currently prints a function that does not exist. An "s" is added. From torchvision.disable_beta_transform_warning() to torchvision.disable_beta_transforms_warning()
Copy link
Collaborator

@pmeier pmeier left a comment

Choose a reason for hiding this comment

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

Thanks @camilodlt! @NicolasHug can we maybe cherry-pick this one or is it too late?

@@ -105,7 +105,7 @@ def _is_tracing():
"this issue: https://github.com/pytorch/vision/issues/6753, and you can also "
"check out https://github.com/pytorch/vision/issues/7319 to learn more about "
"the APIs that we suspect might involve future changes. "
"You can silence this warning by calling torchvision.disable_beta_transform_warning()."
"You can silence this warning by calling torchvision.disable_beta_transforms_warning()."
Copy link
Collaborator

Choose a reason for hiding this comment

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

For reference:

def disable_beta_transforms_warning():

Copy link
Member

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

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

Ah shoot. Thanks a lot @camilodlt . I'll cherry-pick it, but IDK if a new RC will be triggered before the release :/

@NicolasHug NicolasHug merged commit 5850f37 into pytorch:main Mar 7, 2023
@github-actions
Copy link

github-actions bot commented Mar 7, 2023

Hey @NicolasHug!

You merged this PR, but no labels were added. The list of valid labels is available at https://github.com/pytorch/vision/blob/main/.github/process_commit.py

NicolasHug pushed a commit to NicolasHug/vision that referenced this pull request Mar 7, 2023
@pmeier pmeier added the other if you have no clue or if you will manually handle the PR in the release notes label Mar 7, 2023
facebook-github-bot pushed a commit that referenced this pull request Mar 30, 2023
Reviewed By: vmoens

Differential Revision: D44416573

fbshipit-source-id: cb57b893dab3a34601b534079e7b5e7a7b5209c8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla signed other if you have no clue or if you will manually handle the PR in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants