-
Notifications
You must be signed in to change notification settings - Fork 27.7k
TEST: audit uses of maybeWarnsRegex #52025
Copy link
Copy link
Closed
Labels
module: testsIssues related to tests (not the torch.testing module)Issues related to tests (not the torch.testing module)triagedThis issue has been looked at a team member, and triaged and prioritized into an appropriate moduleThis issue has been looked at a team member, and triaged and prioritized into an appropriate module
Metadata
Metadata
Assignees
Labels
module: testsIssues related to tests (not the torch.testing module)Issues related to tests (not the torch.testing module)triagedThis issue has been looked at a team member, and triaged and prioritized into an appropriate moduleThis issue has been looked at a team member, and triaged and prioritized into an appropriate module
Follow on for #47624, which created an
assertWarnsOnceRegexcontext manager to test C-levelTORCH_WARN_ONCE. All the places that currently usemaybeWarnsRegexshould be replaced with theassrtWarnsOnceRegex, and any untestedTORCH_WARN_ONCEcode should be covered by a test.cc @mruberry @VitalyFedyunin @walterddr