Skip to content

Conversation

@VitalyFedyunin
Copy link
Contributor

@VitalyFedyunin VitalyFedyunin commented Oct 29, 2019

Reapply reverted and fix files gen_variable_type.py test_jit.py

#27891 Cleanup testing of _like operators
#27890 Add memory format support to randn_like operator
#27889 Add memory format support to randint_like operator
#27562 Add memory format support to zeros_like operator
#27561 Add memory format support to rand_like operator
#27270 Add memory format support to ones_like operator
#27262 Add memory format support to full_like operator

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@VitalyFedyunin has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot facebook-github-bot added the oncall: jit Add this issue/PR to JIT oncall triage queue label Oct 29, 2019
@VitalyFedyunin VitalyFedyunin marked this pull request as ready for review October 29, 2019 17:27
Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@VitalyFedyunin has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@VitalyFedyunin has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@VitalyFedyunin has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@VitalyFedyunin
Copy link
Contributor Author

xla failure expected
all other failures are unrelated to the PR

@VitalyFedyunin
Copy link
Contributor Author

@pytorchbot retest this please

@ezyang
Copy link
Contributor

ezyang commented Oct 31, 2019

Soooo... what's the fix?

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@VitalyFedyunin has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@kostmo
Copy link
Member

kostmo commented Oct 31, 2019

CircleCI build failures summary

As of commit 51bc068:

  • 0/2 flaky

Here are the reasons each build failed.


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

Please report bugs/suggestions on the GitHub issue tracker.

This comment has been revised 1 time(s).

zdevito pushed a commit to zdevito/ATen that referenced this pull request Oct 31, 2019
…8839)

Summary:
Reapply reverted and fix files `gen_variable_type.py` `test_jit.py`

pytorch/pytorch#27891 Cleanup testing of _like operators
pytorch/pytorch#27890 Add memory format support to randn_like operator
pytorch/pytorch#27889 Add memory format support to randint_like operator
pytorch/pytorch#27562 Add memory format support to zeros_like operator
pytorch/pytorch#27561 Add memory format support to rand_like operator
pytorch/pytorch#27270 Add memory format support to ones_like operator
pytorch/pytorch#27262 Add memory format support to full_like operator
Pull Request resolved: pytorch/pytorch#28839

Test Plan:
Imported from GitHub, without a `Test Plan:` line.

buck test mode/dev //language_technology/neural_mt/os/pytorch_translate/test:test_onnx -- 'test_forced_decoder_export_vocab_reduction \(language_technology\.neural_mt\.os\.pytorch_translate\.test\.test_onnx\.TestONNX\)'

Differential Revision: D18203397

Pulled By: VitalyFedyunin

fbshipit-source-id: eea41cbd4c232cf5a54172b1e1b16b173798f298
@facebook-github-bot
Copy link
Contributor

@VitalyFedyunin merged this pull request in 4bfe2f0.

Comment on lines +522 to 524
auto result = at::empty_like(self, options, optional_memory_format);
return result.random_(0, high, nullptr);
return native::randint(high, self.sizes(), nullptr, options);
Copy link
Contributor

Choose a reason for hiding this comment

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

@VitalyFedyunin, there are two return statements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged oncall: jit Add this issue/PR to JIT oncall triage queue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants