Skip to content

Conversation

glaringlee
Copy link
Contributor

@glaringlee glaringlee commented Aug 20, 2020

Stack from ghstack:

This is to remove the empty pretty_print() since it overrides the impl within Module base which is not as designed here.

Differential Revision: D23244616

glaringlee pushed a commit that referenced this pull request Aug 20, 2020
ghstack-source-id: 33fe00e
Pull Request resolved: #43341
@dr-ci
Copy link

dr-ci bot commented Aug 20, 2020

💊 CI failures summary and remediations

As of commit 04bbb2b (more details on the Dr. CI page):


  • 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 CircleCI build pytorch_linux_backward_compatibility_check_test (1/1)

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

Aug 20 17:55:21 The PR is introducing backward incompatible changes to the operator library. Please contact PyTorch team to confirm whether this change is wanted or not.
Aug 20 17:55:21 processing existing schema:  __str__(__torch__.torch.classes._TorchScriptTesting._StackString _0) -> (str _0) 
Aug 20 17:55:21 processing existing schema:  __init__(__torch__.torch.classes._TorchScriptTesting._PickleTester _0, int[] _1) -> (None _0) 
Aug 20 17:55:21 processing existing schema:  __getstate__(__torch__.torch.classes._TorchScriptTesting._PickleTester _0) -> (int[] _0) 
Aug 20 17:55:21 processing existing schema:  __setstate__(__torch__.torch.classes._TorchScriptTesting._PickleTester _0, int[] _1) -> (None _0) 
Aug 20 17:55:21 processing existing schema:  top(__torch__.torch.classes._TorchScriptTesting._PickleTester _0) -> (int _0) 
Aug 20 17:55:21 processing existing schema:  pop(__torch__.torch.classes._TorchScriptTesting._PickleTester _0) -> (int _0) 
Aug 20 17:55:21 processing existing schema:  get(__torch__.torch.classes._TorchScriptTesting._LiteInterpreterTest _0, Tensor _1) -> (str _0) 
Aug 20 17:55:21 processing existing schema:  __getstate__(__torch__.torch.classes._TorchScriptTesting._LiteInterpreterTest _0) -> (int _0) 
Aug 20 17:55:21 processing existing schema:  __setstate__(__torch__.torch.classes._TorchScriptTesting._LiteInterpreterTest _0, int _1) -> (None _0) 
Aug 20 17:55:21 processing existing schema:  __init__(__torch__.torch.classes.dist_rpc.WorkerInfo _0, str _1, int _2) -> (None _0) 
Aug 20 17:55:21 The PR is introducing backward incompatible changes to the operator library. Please contact PyTorch team to confirm whether this change is wanted or not.  
Aug 20 17:55:21  
Aug 20 17:55:21 Broken ops: [ 
Aug 20 17:55:21 	aten::_compute_linear_combination.out(Tensor input, Tensor coefficients, Tensor(a!) out) -> (Tensor(a!)) 
Aug 20 17:55:21 ] 
Aug 20 17:55:21 =================== sccache compilation log =================== 
Aug 20 17:55:21 + cleanup 
Aug 20 17:55:21 + retcode=1 
Aug 20 17:55:21 + set +x 
Aug 20 17:55:21 =========== If your build fails, please take a look at the log above for possible reasons =========== 
Aug 20 17:55:21 Compile requests                 0 

This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 1 time.

@facebook-github-bot
Copy link
Contributor

@glaringlee merged this pull request in e32d014.

@facebook-github-bot facebook-github-bot deleted the gh/glaringlee/26/head branch August 24, 2020 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants