Skip to content

Updated foreach pointwise ops to support complex and bloat16 on CUDA#51170

Closed
izdeby wants to merge 26 commits intogh/izdeby/79/basefrom
gh/izdeby/79/head
Closed

Updated foreach pointwise ops to support complex and bloat16 on CUDA#51170
izdeby wants to merge 26 commits intogh/izdeby/79/basefrom
gh/izdeby/79/head

Conversation

@izdeby
Copy link
Copy Markdown
Contributor

@izdeby izdeby commented Jan 27, 2021

Stack from ghstack:


Updated foreach pointwise ops to support complex and bloat16 on CUDA

Differential Revision: D26103906

@facebook-github-bot
Copy link
Copy Markdown
Contributor

facebook-github-bot commented Jan 27, 2021

💊 CI failures summary and remediations

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


  • 4/4 failures possibly* introduced in this PR
    • 1/4 non-scanned failure(s)

🕵️ 2 new failures recognized by patterns

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

See CircleCI build pytorch_windows_vs2019_py36_cuda10.1_test1 (1/2)

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

AssertionError: False is not true : Tensors failed to compare as equal!Real parts failed to compare as equal! With rtol=1.3e-06 and atol=1e-05, found 1 element(s) (out of 400) whose difference(s) exceeded the margin of error (including 0 nan comparisons). The greatest difference was 2.09808349609375e-05 (2.9861230850219727 vs. 2.9861440658569336), which occurred at index (18, 9).
  File "C:\Users\circleci\project\build\win_tmp\build\torch\testing\_internal\common_device_type.py", line 292, in instantiated_test
    result = test_fn(self, *args)
  File "C:\Users\circleci\project\build\win_tmp\build\torch\testing\_internal\common_device_type.py", line 266, in test_wrapper
    return test(*args, **kwargs)
  File "test_foreach.py", line 307, in test_pointwise_ops
    self.assertEqual(tensors, expected)
  File "C:\Users\circleci\project\build\win_tmp\build\torch\testing\_internal\common_utils.py", line 1309, in assertEqual
    exact_dtype=exact_dtype, exact_device=exact_device)
  File "C:\Users\circleci\project\build\win_tmp\build\torch\testing\_internal\common_utils.py", line 1275, in assertEqual
    super().assertTrue(result, msg=self._get_assert_msg(msg, debug_msg=debug_msg))
AssertionError: False is not true : Tensors failed to compare as equal!Real parts failed to compare as equal! With rtol=1.3e-06 and atol=1e-05, found 1 element(s) (out of 400) whose difference(s) exceeded the margin of error (including 0 nan comparisons). The greatest difference was 2.09808349609375e-05 (2.9861230850219727 vs. 2.9861440658569336), which occurred at index (18, 9).

----------------------------------------------------------------------
Ran 827 tests in 29.902s

FAILED (failures=1)

Generating XML reports...
Generated XML report: test-reports\dist-gloo\test_foreach\TEST-TestForeachCPU-20210401193543.xml
Generated XML report: test-reports\dist-gloo\test_foreach\TEST-TestForeachCUDA-20210401193543.xml
Traceback (most recent call last):

See CircleCI build pytorch_linux_xenial_cuda10_2_cudnn7_py3_gcc7_test2 (2/2)

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

Apr 01 20:35:50 AssertionError: False is not true : Tensors failed to compare as equal!Real parts failed to compare as equal! With rtol=1.3e-06 and atol=1e-05, found 1 element(s) (out of 400) whose difference(s) exceeded the margin of error (including 0 nan comparisons). The greatest difference was 2.09808349609375e-05 (2.9861230850219727 vs. 2.9861440658569336), which occurred at index (18, 9).
Apr 01 20:35:50   File "/opt/conda/lib/python3.6/site-packages/torch/testing/_internal/common_device_type.py", line 292, in instantiated_test
Apr 01 20:35:50     result = test_fn(self, *args)
Apr 01 20:35:50   File "/opt/conda/lib/python3.6/site-packages/torch/testing/_internal/common_device_type.py", line 266, in test_wrapper
Apr 01 20:35:50     return test(*args, **kwargs)
Apr 01 20:35:50   File "test_foreach.py", line 307, in test_pointwise_ops
Apr 01 20:35:50     self.assertEqual(tensors, expected)
Apr 01 20:35:50   File "/opt/conda/lib/python3.6/site-packages/torch/testing/_internal/common_utils.py", line 1309, in assertEqual
Apr 01 20:35:50     exact_dtype=exact_dtype, exact_device=exact_device)
Apr 01 20:35:50   File "/opt/conda/lib/python3.6/site-packages/torch/testing/_internal/common_utils.py", line 1275, in assertEqual
Apr 01 20:35:50     super().assertTrue(result, msg=self._get_assert_msg(msg, debug_msg=debug_msg))
Apr 01 20:35:50 AssertionError: False is not true : Tensors failed to compare as equal!Real parts failed to compare as equal! With rtol=1.3e-06 and atol=1e-05, found 1 element(s) (out of 400) whose difference(s) exceeded the margin of error (including 0 nan comparisons). The greatest difference was 2.09808349609375e-05 (2.9861230850219727 vs. 2.9861440658569336), which occurred at index (18, 9).
Apr 01 20:35:50 
Apr 01 20:35:50 ----------------------------------------------------------------------
Apr 01 20:35:50 Ran 827 tests in 56.157s
Apr 01 20:35:50 
Apr 01 20:35:50 FAILED (failures=1)
Apr 01 20:35:50 
Apr 01 20:35:50 Generating XML reports...
Apr 01 20:35:50 Generated XML report: test-reports/dist-gloo/test_foreach/TEST-TestForeachCPU-20210401203454.xml
Apr 01 20:35:50 Generated XML report: test-reports/dist-gloo/test_foreach/TEST-TestForeachCUDA-20210401203454.xml
Apr 01 20:35:51 Traceback (most recent call last):

1 failure not recognized by patterns:

Job Step Action
CircleCI pytorch_windows_vs2019_py36_cuda10.1_test2 Install Cudnn 🔁 rerun

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 to the (internal) Dr. CI Users group.

…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

[ghstack-poisoned]
@izdeby izdeby requested review from gchanan, ngimel and zou3519 January 27, 2021 15:27
Iurii Zdebskyi added 5 commits January 27, 2021 09:23
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
izdeby pushed a commit that referenced this pull request Feb 4, 2021
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
Iurii Zdebskyi and others added 17 commits February 18, 2021 12:16
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

[ghstack-poisoned]
…16 on CUDA"


--------
Updated foreach pointwise ops to support complex and bloat16 on CUDA

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

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

Hi @izdeby!

Thank you for your pull request.

We require contributors to sign our Contributor License Agreement, and yours needs attention.

You currently have a record in our system, but the CLA is no longer valid, and will need to be resubmitted.

Process

In order for us to review and merge your suggested changes, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA.

Once the CLA is signed, our tooling will perform checks and validations. Afterwards, the pull request will be tagged with CLA signed. The tagging process may take up to 1 hour after signing. Please give it that time before contacting us about it.

If you have received this in error or have any questions, please contact us at cla@fb.com. Thanks!

@ngimel ngimel removed their request for review May 30, 2021 23:39
@zou3519 zou3519 removed their request for review June 28, 2021 13:31
@github-actions github-actions bot closed this May 12, 2022
@facebook-github-bot facebook-github-bot deleted the gh/izdeby/79/head branch June 11, 2022 14:18
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.

3 participants