Skip to content

Conversation

@dagitses
Copy link
Collaborator

@dagitses dagitses commented May 18, 2022

This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented May 18, 2022

🔗 Helpful links

❌ 1 New Failures

As of commit 79a226b (more details on the Dr. CI page):

Expand to see more
  • 1/1 failures introduced in this PR

🕵️ 1 new failure recognized by patterns

The following CI failures do not appear to be due to upstream breakages

See GitHub Actions build pull / linux-xenial-py3.7-gcc5.4 / test (backwards_compat, 1, 1, linux.2xlarge) (1/1)

Step: "Test" (full log | diagnosis details | 🔁 rerun)

2022-05-31T17:47:28.9321252Z The PR is introduc...m to confirm whether this change is wanted or not.
2022-05-31T17:47:28.9308774Z processing existing schema:  text(__torch__.torch.classes.profiling.SourceRef _0) -> (str _0)
2022-05-31T17:47:28.9310180Z processing existing schema:  count(__torch__.torch.classes.profiling.InstructionStats _0) -> (int _0)
2022-05-31T17:47:28.9311418Z processing existing schema:  duration_ns(__torch__.torch.classes.profiling.InstructionStats _0) -> (int _0)
2022-05-31T17:47:28.9312452Z processing existing schema:  source(__torch__.torch.classes.profiling.SourceStats _0) -> (__torch__.torch.classes.profiling.SourceRef _0)
2022-05-31T17:47:28.9314401Z processing existing schema:  line_map(__torch__.torch.classes.profiling.SourceStats _0) -> (Dict(int, __torch__.torch.classes.profiling.InstructionStats) _0)
2022-05-31T17:47:28.9315342Z processing existing schema:  __init__(__torch__.torch.classes.profiling._ScriptProfile _0) -> (NoneType _0)
2022-05-31T17:47:28.9316563Z processing existing schema:  enable(__torch__.torch.classes.profiling._ScriptProfile _0) -> (NoneType _0)
2022-05-31T17:47:28.9317691Z processing existing schema:  disable(__torch__.torch.classes.profiling._ScriptProfile _0) -> (NoneType _0)
2022-05-31T17:47:28.9319501Z processing existing schema:  _dump_stats(__torch__.torch.classes.profiling._ScriptProfile _0) -> (__torch__.torch.classes.profiling.SourceStats[] _0)
2022-05-31T17:47:28.9321010Z processing existing schema:  __init__(__torch__.torch.classes.dist_rpc.WorkerInfo _0, str _1, int _2) -> (NoneType _0)
2022-05-31T17:47:28.9321252Z The PR is introducing backward incompatible changes to the operator library. Please contact PyTorch team to confirm whether this change is wanted or not. 
2022-05-31T17:47:28.9321279Z 
2022-05-31T17:47:28.9321855Z Broken ops: [
2022-05-31T17:47:28.9322133Z 	prims::uniform(int[] shape, *, Scalar low, Scalar high, int dtype, Device device) -> (Tensor)
2022-05-31T17:47:28.9323933Z 	prims::empty_strided(int[] shape, int[] strides, *, int dtype, Device device, bool requires_grad) -> (Tensor)
2022-05-31T17:47:28.9324259Z 	prims::var(Tensor inp, int[]? dims, *, int correction, int? output_dtype=None) -> (Tensor)
2022-05-31T17:47:28.9324416Z 	prims::minium_value(int dtype) -> (Scalar)
2022-05-31T17:47:28.9324598Z 	prims::maximum_value(int dtype) -> (Scalar)
2022-05-31T17:47:28.9324735Z 	prims::item(Tensor a) -> (Scalar)
2022-05-31T17:47:28.9324917Z 	prims::where(Tensor pred, Tensor a, Tensor b) -> (Tensor)
2022-05-31T17:47:28.9325080Z 	prims::cat(Tensor[] tensors, int dim) -> (Tensor)

This comment was automatically generated by Dr. CI (expand for details).

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
Michael Andreas Dagitses added 2 commits May 18, 2022 08:38
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
Michael Andreas Dagitses added 2 commits May 18, 2022 08:57
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
Michael Andreas Dagitses added 5 commits May 18, 2022 09:17
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
Michael Andreas Dagitses added 2 commits May 18, 2022 12:02
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
Michael Andreas Dagitses added 11 commits May 20, 2022 19:33
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
Michael Andreas Dagitses added 2 commits May 23, 2022 09:58
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
Michael Andreas Dagitses added 6 commits May 23, 2022 10:36
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
This will allow a great refactoring to eliminate duplication in
aten.bzl.

Differential Revision: [D36480810](https://our.internmc.facebook.com/intern/diff/D36480810/)

**NOTE FOR REVIEWERS**: This PR has internal Facebook specific changes or comments, please review them on [Phabricator](https://our.internmc.facebook.com/intern/diff/D36480810/)!

[ghstack-poisoned]
@kit1980
Copy link
Contributor

kit1980 commented Jun 6, 2022

Please rebase on master, several workflows were removed/added.

@dagitses dagitses closed this Jun 16, 2022
@dagitses dagitses deleted the gh/dagitses/337/head branch June 16, 2022 08:30
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.

4 participants