New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

rename default dim order as contiguous dim order #2157

Closed

Gasoonjia wants to merge 2 commits into pytorch:main from Gasoonjia:export-D54285070

Contributor

Gasoonjia commented Feb 28, 2024

Differential Revision: D54285070

pytorch-bot bot commented Feb 28, 2024 •

edited

Loading

🔗 Helpful Links

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

📄 Preview Python docs built from this PR

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

❌ 6 New Failures, 1 Unrelated Failure

As of commit 7ee7e65 with merge base 8532e79 ():

NEW FAILURES - The following jobs have failed:

Lint / lintrunner / linux-job (gh)
>>> Lint for runtime/core/exec_aten/testing_util/tensor_factory.h:
pull / test-llama-runner-linux (fp16, buck2) / linux-job (gh)
RuntimeError: Failed to compile /tmp/tmpm0b67pgp/data.json to /tmp/tmpm0b67pgp/data.pte. Set ET_EXIR_SAVE_FLATC_INPUTS_ON_FAILURE=1 to save input files on failure.
pull / test-llama-runner-linux (fp16, cmake) / linux-job (gh)
RuntimeError: Failed to compile /tmp/tmpi_duo2go/data.json to /tmp/tmpi_duo2go/data.pte. Set ET_EXIR_SAVE_FLATC_INPUTS_ON_FAILURE=1 to save input files on failure.
pull / test-llama-runner-linux (fp32, buck2) / linux-job (gh)
RuntimeError: Failed to compile /tmp/tmpuzea9sat/data.json to /tmp/tmpuzea9sat/data.pte. Set ET_EXIR_SAVE_FLATC_INPUTS_ON_FAILURE=1 to save input files on failure.
pull / test-llama-runner-linux (fp32, cmake) / linux-job (gh)
RuntimeError: Command docker exec -t 312262d5b43109bbb2bbe925cc5ecf0b6bf22cd575ae5d4428a955d1c89bde34 /exec failed with exit code 1
pull / unittest / linux (buck2) / linux-job (gh)
RuntimeError: Command docker exec -t 5054ee791064ff15fb390aac3317a6fcb20e3e1feb083520e206951b6f0d19d8 /exec failed with exit code 1

BROKEN TRUNK - The following job failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

pull / unittest / macos (buck2) / macos-job (gh)
RuntimeError: Command bash /Users/ec2-user/runner/_work/_temp/exec_script failed with exit code 1

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

facebook-github-bot added the CLA Signed label

Contributor

facebook-github-bot commented Feb 28, 2024

This pull request was exported from Phabricator. Differential Revision: D54285070

facebook-github-bot added the fb-exported label

Gasoonjia force-pushed the export-D54285070 branch from 37e0867 to 2b80aec Compare

March 5, 2024 06:21

Contributor

facebook-github-bot commented Mar 5, 2024

This pull request was exported from Phabricator. Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

5f5bc4b

Summary:

bypass-github-export-checks

Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

dcb2031

Summary:

bypass-github-export-checks

Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

b5da875

Summary:

bypass-github-export-checks

Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

2f0db86

Summary:

bypass-github-export-checks

Differential Revision: D54285070

Gasoonjia force-pushed the export-D54285070 branch from 2b80aec to 775d8b0 Compare

March 5, 2024 21:18

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

775d8b0

Summary:

bypass-github-export-checks

Differential Revision: D54285070

Contributor

facebook-github-bot commented Mar 5, 2024

This pull request was exported from Phabricator. Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

035784b

Summary:

bypass-github-export-checks

Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

3634d39

Summary:

bypass-github-export-checks

Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

3abe0b6

Summary:

bypass-github-export-checks

Reviewed By: digantdesai

Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

c240dfb

Summary:

bypass-github-export-checks

Reviewed By: digantdesai

Differential Revision: D54285070

Gasoonjia force-pushed the export-D54285070 branch from 775d8b0 to c240dfb Compare

March 7, 2024 23:58

Contributor

facebook-github-bot commented Mar 8, 2024

This pull request was exported from Phabricator. Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

6981a68

Summary:

bypass-github-export-checks

Reviewed By: digantdesai

Differential Revision: D54285070

Gasoonjia force-pushed the export-D54285070 branch from c240dfb to 6981a68 Compare

March 11, 2024 06:30

Contributor

facebook-github-bot commented Mar 11, 2024

This pull request was exported from Phabricator. Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

c0b5f5f

Summary:

bypass-github-export-checks

Reviewed By: digantdesai

Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

4240c5f

Summary:

bypass-github-export-checks

Reviewed By: digantdesai

Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

64aa335

Summary:

bypass-github-export-checks

Reviewed By: digantdesai

Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

d53e4e8

Summary:

bypass-github-export-checks

Reviewed By: digantdesai

Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

4b006a4

Summary:

bypass-github-export-checks

Reviewed By: digantdesai

Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

dc8ff94

Summary:

bypass-github-export-checks

Reviewed By: digantdesai

Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

0e2ff85

Summary:

bypass-github-export-checks

Reviewed By: digantdesai

Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

d7f7191

Summary:

bypass-github-export-checks

Reviewed By: digantdesai

Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

e81fbaa

Summary:

bypass-github-export-checks

Reviewed By: digantdesai

Differential Revision: D54285070

Gasoonjia pushed a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

4d1ff1e

Summary:
Pull Request resolved: pytorch#2157

bypass-github-export-checks

Differential Revision: https://internalfb.com/D54285070

Gasoonjia pushed a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

3531a3a

Summary:
Pull Request resolved: pytorch#2157

bypass-github-export-checks

Differential Revision: https://internalfb.com/D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

Summary:

bypass-github-export-checks

Reviewed By: digantdesai

Differential Revision: D54285070

Gasoonjia force-pushed the export-D54285070 branch from 34eb93e to a02217e Compare

March 20, 2024 23:10

Contributor

facebook-github-bot commented Mar 20, 2024

This pull request was exported from Phabricator. Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

3d026e3

Summary:

bypass-github-export-checks

Reviewed By: digantdesai

Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

7d9b741

Summary: Pull Request resolved: pytorch#2157

Reviewed By: digantdesai

Differential Revision: D54285070

Gasoonjia pushed a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

4f811b1

Summary:
Pull Request resolved: pytorch#2157

bypass-github-export-checks

Differential Revision: https://internalfb.com/D54285070

Gasoonjia force-pushed the export-D54285070 branch from a02217e to 60d5538 Compare

March 20, 2024 23:50

Contributor

facebook-github-bot commented Mar 20, 2024

This pull request was exported from Phabricator. Differential Revision: D54285070

Gasoonjia pushed a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

7131a20

Summary:
Pull Request resolved: pytorch#2157

bypass-github-export-checks

Differential Revision: https://internalfb.com/D54285070

Gasoonjia pushed a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

a332444

Summary:
Pull Request resolved: pytorch#2157

bypass-github-export-checks

Differential Revision: https://internalfb.com/D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

5f7015f

Summary: Pull Request resolved: pytorch#2157

Reviewed By: digantdesai

Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

70eba33

Summary: Pull Request resolved: pytorch#2157

Reviewed By: digantdesai

Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

4989c37

Summary: Pull Request resolved: pytorch#2157

Reviewed By: digantdesai

Differential Revision: D54285070

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

ab7c28e

Summary: Pull Request resolved: pytorch#2157

Reviewed By: digantdesai

Differential Revision: D54285070

larryliu0820 approved these changes

View reviewed changes

Gasoonjia added a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

a62b201

Summary: Pull Request resolved: pytorch#2157

Reviewed By: digantdesai, larryliu0820

Differential Revision: D54285070

Gasoonjia force-pushed the export-D54285070 branch from 60d5538 to a62b201 Compare

March 22, 2024 19:26

Contributor

facebook-github-bot commented Mar 22, 2024

This pull request was exported from Phabricator. Differential Revision: D54285070

Songhao Jia and others added 2 commits

March 22, 2024 16:45


          make utils support empty dim order (pytorch#2142)

16a97b4

Summary:
Pull Request resolved: pytorch#2142

This update makes util function support empty dim order, to make the empty dim order behave the same as empty memory format (preserve_format).
bypass-github-export-checks

Differential Revision: https://www.internalfb.com/diff/D54236386?entry_point=27


          rename default dim order as contiguous dim order (pytorch#2157)

7ee7e65

Summary: Pull Request resolved: pytorch#2157

Reviewed By: digantdesai, larryliu0820

Differential Revision: D54285070

Gasoonjia pushed a commit to Gasoonjia/executorch-1 that referenced this pull request


          rename default dim order as contiguous dim order (pytorch#2157)

d1e04f6

Summary:
Pull Request resolved: pytorch#2157

bypass-github-export-checks

Differential Revision: https://internalfb.com/D54285070

Contributor

facebook-github-bot commented Mar 22, 2024

This pull request was exported from Phabricator. Differential Revision: D54285070

Gasoonjia force-pushed the export-D54285070 branch from a62b201 to 7ee7e65 Compare

March 22, 2024 23:47

facebook-github-bot closed this in

102614a

facebook-github-bot added the Merged label

Contributor

facebook-github-bot commented Mar 24, 2024

This pull request has been merged in 102614a.

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

Labels

CLA Signed fb-exported Merged