Skip to content

Conversation

@pmeier
Copy link
Collaborator

@pmeier pmeier commented Oct 10, 2022

Per title. Deprecation happened on Feb 25, 2022 in c6f1bbc, which made it into the 1.12 release. Since it is now 227 days later and the next release will be 1.14, this removal complies with the BC policy.

For internal users nothing should change as long as they imported everything from torch.testing._internal. A quick search through the repo did not turn up anything to the contrary.

@pmeier pmeier added module: testing Issues related to the torch.testing module (not tests) topic: bc breaking topic category topic: deprecation topic category labels Oct 10, 2022
@pmeier pmeier requested a review from mruberry October 10, 2022 11:05
@pytorch-bot
Copy link

pytorch-bot bot commented Oct 10, 2022

🔗 Helpful Links

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

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

✅ No Failures

As of commit b5c9386:
💚 Looks good so far! There are no failures yet. 💚

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

@pmeier pmeier requested a review from ngimel as a code owner October 10, 2022 11:43
@pmeier
Copy link
Collaborator Author

pmeier commented Oct 10, 2022

It seems there is more to do here. I'll convert to draft for now and ping again when CI is green.

@pmeier pmeier marked this pull request as draft October 10, 2022 16:07
@pmeier pmeier removed request for mruberry and ngimel October 10, 2022 16:07
@pmeier pmeier added ciflow/trunk Trigger trunk jobs on your pull request ciflow/periodic Trigger jobs ran periodically on master (periodic.yml) on the PR labels Oct 12, 2022
pmeier added a commit that referenced this pull request Oct 28, 2022
…y in torch.testing"


_Redo of #86586 with all BC breaking changes granularly placed into separate commits._

---

Per title. Deprecation happened on Feb 25, 2022 in c6f1bbc, which made it into the 1.12 release. Since it is now 245 days later and the next release will be 1.14, the removals later in the stack comply with the [BC policy](https://github.com/pytorch/pytorch/wiki/PyTorch's-Python-Frontend-Backward-and-Forward-Compatibility-Policy#minimizing-the-disruption-of-bc-breaking-changes).

[ghstack-poisoned]
pmeier added a commit that referenced this pull request Oct 28, 2022
_Redo of #86586 with all BC breaking changes granularly placed into separate commits._

---

Per title. Deprecation happened on Feb 25, 2022 in c6f1bbc, which made it into the 1.12 release. Since it is now 245 days later and the next release will be 1.14, the removals later in the stack comply with the [BC policy](https://github.com/pytorch/pytorch/wiki/PyTorch's-Python-Frontend-Backward-and-Forward-Compatibility-Policy#minimizing-the-disruption-of-bc-breaking-changes).

[ghstack-poisoned]
pmeier added a commit that referenced this pull request Nov 2, 2022
…y in torch.testing"


_Redo of #86586 with all BC breaking changes granularly placed into separate commits._

---

Per title. Deprecation happened on Feb 25, 2022 in c6f1bbc, which made it into the 1.12 release. Since it is now 245 days later and the next release will be 1.14, the removals later in the stack comply with the [BC policy](https://github.com/pytorch/pytorch/wiki/PyTorch's-Python-Frontend-Backward-and-Forward-Compatibility-Policy#minimizing-the-disruption-of-bc-breaking-changes).

[ghstack-poisoned]
pmeier added a commit that referenced this pull request Nov 2, 2022
_Redo of #86586 with all BC breaking changes granularly placed into separate commits._

---

Per title. Deprecation happened on Feb 25, 2022 in c6f1bbc, which made it into the 1.12 release. Since it is now 245 days later and the next release will be 1.14, the removals later in the stack comply with the [BC policy](https://github.com/pytorch/pytorch/wiki/PyTorch's-Python-Frontend-Backward-and-Forward-Compatibility-Policy#minimizing-the-disruption-of-bc-breaking-changes).

[ghstack-poisoned]
@pmeier
Copy link
Collaborator Author

pmeier commented Nov 2, 2022

I've added a stack at #87969 that splits off each BC breaking change into a different commit. I've copied all the highlights I added here to the stack. @mruberry I also addressed your comments in the stack rather than doing it here.

@pmeier pmeier closed this Nov 2, 2022
@pmeier pmeier deleted the remove-deprecated branch November 2, 2022 10:29
pytorchmergebot pushed a commit that referenced this pull request Nov 2, 2022
_Redo of #86586 with all BC breaking changes granularly placed into separate commits._

---

Per title. Deprecation happened on Feb 25, 2022 in c6f1bbc, which made it into the 1.12 release. Since it is now 245 days later and the next release will be 1.14, the removals later in the stack comply with the [BC policy](https://github.com/pytorch/pytorch/wiki/PyTorch's-Python-Frontend-Backward-and-Forward-Compatibility-Policy#minimizing-the-disruption-of-bc-breaking-changes).
Pull Request resolved: #87969
Approved by: https://github.com/mruberry
pytorchmergebot pushed a commit that referenced this pull request Nov 2, 2022
pytorchmergebot pushed a commit that referenced this pull request Nov 2, 2022
pytorchmergebot pushed a commit that referenced this pull request Nov 2, 2022
pytorchmergebot pushed a commit that referenced this pull request Nov 2, 2022
pytorchmergebot pushed a commit that referenced this pull request Nov 2, 2022
See #87969 or #86586 for the reasoning.

Pull Request resolved: #87974
Approved by: https://github.com/mruberry
kulinseth pushed a commit to kulinseth/pytorch that referenced this pull request Nov 5, 2022
…#87969)

_Redo of pytorch#86586 with all BC breaking changes granularly placed into separate commits._

---

Per title. Deprecation happened on Feb 25, 2022 in c6f1bbc, which made it into the 1.12 release. Since it is now 245 days later and the next release will be 1.14, the removals later in the stack comply with the [BC policy](https://github.com/pytorch/pytorch/wiki/PyTorch's-Python-Frontend-Backward-and-Forward-Compatibility-Policy#minimizing-the-disruption-of-bc-breaking-changes).
Pull Request resolved: pytorch#87969
Approved by: https://github.com/mruberry
kulinseth pushed a commit to kulinseth/pytorch that referenced this pull request Nov 5, 2022
kulinseth pushed a commit to kulinseth/pytorch that referenced this pull request Nov 5, 2022
kulinseth pushed a commit to kulinseth/pytorch that referenced this pull request Nov 5, 2022
kulinseth pushed a commit to kulinseth/pytorch that referenced this pull request Nov 5, 2022
kulinseth pushed a commit to kulinseth/pytorch that referenced this pull request Nov 5, 2022
kulinseth pushed a commit to kulinseth/pytorch that referenced this pull request Dec 10, 2022
…#87969)

_Redo of pytorch#86586 with all BC breaking changes granularly placed into separate commits._

---

Per title. Deprecation happened on Feb 25, 2022 in c6f1bbc, which made it into the 1.12 release. Since it is now 245 days later and the next release will be 1.14, the removals later in the stack comply with the [BC policy](https://github.com/pytorch/pytorch/wiki/PyTorch's-Python-Frontend-Backward-and-Forward-Compatibility-Policy#minimizing-the-disruption-of-bc-breaking-changes).
Pull Request resolved: pytorch#87969
Approved by: https://github.com/mruberry
kulinseth pushed a commit to kulinseth/pytorch that referenced this pull request Dec 10, 2022
kulinseth pushed a commit to kulinseth/pytorch that referenced this pull request Dec 10, 2022
kulinseth pushed a commit to kulinseth/pytorch that referenced this pull request Dec 10, 2022
kulinseth pushed a commit to kulinseth/pytorch that referenced this pull request Dec 10, 2022
kulinseth pushed a commit to kulinseth/pytorch that referenced this pull request Dec 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/periodic Trigger jobs ran periodically on master (periodic.yml) on the PR ciflow/trunk Trigger trunk jobs on your pull request module: testing Issues related to the torch.testing module (not tests) open source topic: bc breaking topic category topic: deprecation topic category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants