Add input size validation to Module.execute by keyprocedure · Pull Request #10701 · pytorch/executorch

keyprocedure · 2025-05-05T23:56:26Z

Summary

Adds input size validation to Module.execute to prevent possible silent memory corruption when too many EValue inputs are passed.

Fixes#10510

Test plan

Added unit test TestExecuteWithTooManyInputs
Verified by successfully running all module_test.cpp tests, except TestPTD (did not have access to ModuleLinear.ptd)
To run locally:
- Bypass is_fbcode guard in targets.bzl and redirect test file paths to use a locally exported ModuleAdd.pte file
- Build and run tests via:
```
buck2 build //extension/module/test:test
buck2 run //extension/module/test:test
```

pytorch-bot · 2025-05-05T23:56:29Z

🔗 Helpful Links

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

📄 Preview Python docs built from this PR

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

❌ 1 New Failure, 1 Unrelated Failure

As of commit bb83697 with merge base d069d65 ():

NEW FAILURE - The following job has failed:

pull / unittest-editable / linux / linux-job (gh)
extension/llm/modules/test/test_position_embeddings.py::TiledTokenPositionalEmbeddingTest::test_tiled_token_positional_embedding_et

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

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

pull / test-llava-runner-linux / linux-job (gh) (trunk failure)
test_llava_export

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

keyprocedure · 2025-05-05T23:58:55Z

@pytorchbot label "topic: not user facing"

keyprocedure · 2025-05-15T02:14:22Z

@pytorchbot label "release notes: none"

keyprocedure · 2025-05-19T19:05:59Z

Hi @shoumikhin, the CI failures are unrelated to this PR; they're due totest_llava_export and test_position_embeddings.py. Could you please merge when you get a chance?

### Summary Adds input size validation to `Module.execute` to prevent possible silent memory corruption when too many EValue inputs are passed. Fixespytorch#10510 ### Test plan - Added unit test `TestExecuteWithTooManyInputs` - Verified by successfully running all `module_test.cpp` tests, except `TestPTD` (did not have access to `ModuleLinear.ptd`) - To run locally: - Bypass `is_fbcode` guard in `targets.bzl` and redirect test file paths to use a locally exported `ModuleAdd.pte` file - Build and run tests via: ``` buck2 build //extension/module/test:test buck2 run //extension/module/test:test --------- Co-authored-by: Anthony Shoumikhin <anthony@shoumikh.in>

keyprocedure · 2025-05-05T23:56:26Z

Summary

Adds input size validation to Module.execute to prevent possible silent memory corruption when too many EValue inputs are passed.

Fixes#10510

Test plan

Added unit test TestExecuteWithTooManyInputs
Verified by successfully running all module_test.cpp tests, except TestPTD (did not have access to ModuleLinear.ptd)
To run locally:
- Bypass is_fbcode guard in targets.bzl and redirect test file paths to use a locally exported ModuleAdd.pte file
- Build and run tests via:
```
buck2 build //extension/module/test:test
buck2 run //extension/module/test:test
```

pytorch-bot · 2025-05-05T23:56:29Z

🔗 Helpful Links

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

📄 Preview Python docs built from this PR

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

❌ 1 New Failure, 1 Unrelated Failure

As of commit bb83697 with merge base d069d65 ():

NEW FAILURE - The following job has failed:

pull / unittest-editable / linux / linux-job (gh)
extension/llm/modules/test/test_position_embeddings.py::TiledTokenPositionalEmbeddingTest::test_tiled_token_positional_embedding_et

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

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

pull / test-llava-runner-linux / linux-job (gh) (trunk failure)
test_llava_export

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

keyprocedure · 2025-05-05T23:58:55Z

@pytorchbot label "topic: not user facing"

keyprocedure · 2025-05-15T02:14:22Z

@pytorchbot label "release notes: none"

keyprocedure · 2025-05-19T19:05:59Z

Hi @shoumikhin, the CI failures are unrelated to this PR; they're due totest_llava_export and test_position_embeddings.py. Could you please merge when you get a chance?

### Summary Adds input size validation to `Module.execute` to prevent possible silent memory corruption when too many EValue inputs are passed. Fixespytorch#10510 ### Test plan - Added unit test `TestExecuteWithTooManyInputs` - Verified by successfully running all `module_test.cpp` tests, except `TestPTD` (did not have access to `ModuleLinear.ptd`) - To run locally: - Bypass `is_fbcode` guard in `targets.bzl` and redirect test file paths to use a locally exported `ModuleAdd.pte` file - Build and run tests via: ``` buck2 build //extension/module/test:test buck2 run //extension/module/test:test --------- Co-authored-by: Anthony Shoumikhin <anthony@shoumikh.in>

keyprocedure · 2025-05-05T23:56:26Z

Summary

Adds input size validation to Module.execute to prevent possible silent memory corruption when too many EValue inputs are passed.

Fixes#10510

Test plan

Added unit test TestExecuteWithTooManyInputs
Verified by successfully running all module_test.cpp tests, except TestPTD (did not have access to ModuleLinear.ptd)
To run locally:
- Bypass is_fbcode guard in targets.bzl and redirect test file paths to use a locally exported ModuleAdd.pte file
- Build and run tests via:
```
buck2 build //extension/module/test:test
buck2 run //extension/module/test:test
```

pytorch-bot · 2025-05-05T23:56:29Z

🔗 Helpful Links

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

📄 Preview Python docs built from this PR

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

❌ 1 New Failure, 1 Unrelated Failure

As of commit bb83697 with merge base d069d65 ():

NEW FAILURE - The following job has failed:

pull / unittest-editable / linux / linux-job (gh)
extension/llm/modules/test/test_position_embeddings.py::TiledTokenPositionalEmbeddingTest::test_tiled_token_positional_embedding_et

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

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

pull / test-llava-runner-linux / linux-job (gh) (trunk failure)
test_llava_export

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

keyprocedure · 2025-05-05T23:58:55Z

@pytorchbot label "topic: not user facing"

keyprocedure · 2025-05-15T02:14:22Z

@pytorchbot label "release notes: none"

keyprocedure · 2025-05-19T19:05:59Z

Hi @shoumikhin, the CI failures are unrelated to this PR; they're due totest_llava_export and test_position_embeddings.py. Could you please merge when you get a chance?

### Summary Adds input size validation to `Module.execute` to prevent possible silent memory corruption when too many EValue inputs are passed. Fixespytorch#10510 ### Test plan - Added unit test `TestExecuteWithTooManyInputs` - Verified by successfully running all `module_test.cpp` tests, except `TestPTD` (did not have access to `ModuleLinear.ptd`) - To run locally: - Bypass `is_fbcode` guard in `targets.bzl` and redirect test file paths to use a locally exported `ModuleAdd.pte` file - Build and run tests via: ``` buck2 build //extension/module/test:test buck2 run //extension/module/test:test --------- Co-authored-by: Anthony Shoumikhin <anthony@shoumikh.in>

keyprocedure · 2025-05-05T23:56:26Z

Summary

Adds input size validation to Module.execute to prevent possible silent memory corruption when too many EValue inputs are passed.

Fixes#10510

Test plan

Added unit test TestExecuteWithTooManyInputs
Verified by successfully running all module_test.cpp tests, except TestPTD (did not have access to ModuleLinear.ptd)
To run locally:
- Bypass is_fbcode guard in targets.bzl and redirect test file paths to use a locally exported ModuleAdd.pte file
- Build and run tests via:
```
buck2 build //extension/module/test:test
buck2 run //extension/module/test:test
```

pytorch-bot · 2025-05-05T23:56:29Z

🔗 Helpful Links

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

📄 Preview Python docs built from this PR

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

❌ 1 New Failure, 1 Unrelated Failure

As of commit bb83697 with merge base d069d65 ():

NEW FAILURE - The following job has failed:

pull / unittest-editable / linux / linux-job (gh)
extension/llm/modules/test/test_position_embeddings.py::TiledTokenPositionalEmbeddingTest::test_tiled_token_positional_embedding_et

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

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

pull / test-llava-runner-linux / linux-job (gh) (trunk failure)
test_llava_export

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

keyprocedure · 2025-05-05T23:58:55Z

@pytorchbot label "topic: not user facing"

keyprocedure · 2025-05-15T02:14:22Z

@pytorchbot label "release notes: none"

keyprocedure · 2025-05-19T19:05:59Z

Hi @shoumikhin, the CI failures are unrelated to this PR; they're due totest_llava_export and test_position_embeddings.py. Could you please merge when you get a chance?

### Summary Adds input size validation to `Module.execute` to prevent possible silent memory corruption when too many EValue inputs are passed. Fixespytorch#10510 ### Test plan - Added unit test `TestExecuteWithTooManyInputs` - Verified by successfully running all `module_test.cpp` tests, except `TestPTD` (did not have access to `ModuleLinear.ptd`) - To run locally: - Bypass `is_fbcode` guard in `targets.bzl` and redirect test file paths to use a locally exported `ModuleAdd.pte` file - Build and run tests via: ``` buck2 build //extension/module/test:test buck2 run //extension/module/test:test --------- Co-authored-by: Anthony Shoumikhin <anthony@shoumikh.in>

keyprocedure · 2025-05-05T23:56:26Z

Summary

Adds input size validation to Module.execute to prevent possible silent memory corruption when too many EValue inputs are passed.

Fixes#10510

Test plan

Added unit test TestExecuteWithTooManyInputs
Verified by successfully running all module_test.cpp tests, except TestPTD (did not have access to ModuleLinear.ptd)
To run locally:
- Bypass is_fbcode guard in targets.bzl and redirect test file paths to use a locally exported ModuleAdd.pte file
- Build and run tests via:
```
buck2 build //extension/module/test:test
buck2 run //extension/module/test:test
```

pytorch-bot · 2025-05-05T23:56:29Z

🔗 Helpful Links

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

📄 Preview Python docs built from this PR

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

❌ 1 New Failure, 1 Unrelated Failure

As of commit bb83697 with merge base d069d65 ():

NEW FAILURE - The following job has failed:

pull / unittest-editable / linux / linux-job (gh)
extension/llm/modules/test/test_position_embeddings.py::TiledTokenPositionalEmbeddingTest::test_tiled_token_positional_embedding_et

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

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

pull / test-llava-runner-linux / linux-job (gh) (trunk failure)
test_llava_export

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

keyprocedure · 2025-05-05T23:58:55Z

@pytorchbot label "topic: not user facing"

keyprocedure · 2025-05-15T02:14:22Z

@pytorchbot label "release notes: none"

keyprocedure · 2025-05-19T19:05:59Z

Hi @shoumikhin, the CI failures are unrelated to this PR; they're due totest_llava_export and test_position_embeddings.py. Could you please merge when you get a chance?

### Summary Adds input size validation to `Module.execute` to prevent possible silent memory corruption when too many EValue inputs are passed. Fixespytorch#10510 ### Test plan - Added unit test `TestExecuteWithTooManyInputs` - Verified by successfully running all `module_test.cpp` tests, except `TestPTD` (did not have access to `ModuleLinear.ptd`) - To run locally: - Bypass `is_fbcode` guard in `targets.bzl` and redirect test file paths to use a locally exported `ModuleAdd.pte` file - Build and run tests via: ``` buck2 build //extension/module/test:test buck2 run //extension/module/test:test --------- Co-authored-by: Anthony Shoumikhin <anthony@shoumikh.in>

keyprocedure · 2025-05-05T23:56:26Z

Summary

Adds input size validation to Module.execute to prevent possible silent memory corruption when too many EValue inputs are passed.

Fixes#10510

Test plan

Added unit test TestExecuteWithTooManyInputs
Verified by successfully running all module_test.cpp tests, except TestPTD (did not have access to ModuleLinear.ptd)
To run locally:
- Bypass is_fbcode guard in targets.bzl and redirect test file paths to use a locally exported ModuleAdd.pte file
- Build and run tests via:
```
buck2 build //extension/module/test:test
buck2 run //extension/module/test:test
```

pytorch-bot · 2025-05-05T23:56:29Z

🔗 Helpful Links

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

📄 Preview Python docs built from this PR

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

❌ 1 New Failure, 1 Unrelated Failure

As of commit bb83697 with merge base d069d65 ():

NEW FAILURE - The following job has failed:

pull / unittest-editable / linux / linux-job (gh)
extension/llm/modules/test/test_position_embeddings.py::TiledTokenPositionalEmbeddingTest::test_tiled_token_positional_embedding_et

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

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

pull / test-llava-runner-linux / linux-job (gh) (trunk failure)
test_llava_export

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

keyprocedure · 2025-05-05T23:58:55Z

@pytorchbot label "topic: not user facing"

keyprocedure · 2025-05-15T02:14:22Z

@pytorchbot label "release notes: none"

keyprocedure · 2025-05-19T19:05:59Z

Hi @shoumikhin, the CI failures are unrelated to this PR; they're due totest_llava_export and test_position_embeddings.py. Could you please merge when you get a chance?

### Summary Adds input size validation to `Module.execute` to prevent possible silent memory corruption when too many EValue inputs are passed. Fixespytorch#10510 ### Test plan - Added unit test `TestExecuteWithTooManyInputs` - Verified by successfully running all `module_test.cpp` tests, except `TestPTD` (did not have access to `ModuleLinear.ptd`) - To run locally: - Bypass `is_fbcode` guard in `targets.bzl` and redirect test file paths to use a locally exported `ModuleAdd.pte` file - Build and run tests via: ``` buck2 build //extension/module/test:test buck2 run //extension/module/test:test --------- Co-authored-by: Anthony Shoumikhin <anthony@shoumikh.in>

keyprocedure · 2025-05-05T23:56:26Z

Summary

Adds input size validation to Module.execute to prevent possible silent memory corruption when too many EValue inputs are passed.

Fixes#10510

Test plan

Added unit test TestExecuteWithTooManyInputs
Verified by successfully running all module_test.cpp tests, except TestPTD (did not have access to ModuleLinear.ptd)
To run locally:
- Bypass is_fbcode guard in targets.bzl and redirect test file paths to use a locally exported ModuleAdd.pte file
- Build and run tests via:
```
buck2 build //extension/module/test:test
buck2 run //extension/module/test:test
```

pytorch-bot · 2025-05-05T23:56:29Z

🔗 Helpful Links

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

📄 Preview Python docs built from this PR

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

❌ 1 New Failure, 1 Unrelated Failure

As of commit bb83697 with merge base d069d65 ():

NEW FAILURE - The following job has failed:

pull / unittest-editable / linux / linux-job (gh)
extension/llm/modules/test/test_position_embeddings.py::TiledTokenPositionalEmbeddingTest::test_tiled_token_positional_embedding_et

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

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

pull / test-llava-runner-linux / linux-job (gh) (trunk failure)
test_llava_export

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

keyprocedure · 2025-05-05T23:58:55Z

@pytorchbot label "topic: not user facing"

keyprocedure · 2025-05-15T02:14:22Z

@pytorchbot label "release notes: none"

keyprocedure · 2025-05-19T19:05:59Z

Hi @shoumikhin, the CI failures are unrelated to this PR; they're due totest_llava_export and test_position_embeddings.py. Could you please merge when you get a chance?

### Summary Adds input size validation to `Module.execute` to prevent possible silent memory corruption when too many EValue inputs are passed. Fixespytorch#10510 ### Test plan - Added unit test `TestExecuteWithTooManyInputs` - Verified by successfully running all `module_test.cpp` tests, except `TestPTD` (did not have access to `ModuleLinear.ptd`) - To run locally: - Bypass `is_fbcode` guard in `targets.bzl` and redirect test file paths to use a locally exported `ModuleAdd.pte` file - Build and run tests via: ``` buck2 build //extension/module/test:test buck2 run //extension/module/test:test --------- Co-authored-by: Anthony Shoumikhin <anthony@shoumikh.in>

keyprocedure · 2025-05-05T23:56:26Z

Summary

Adds input size validation to Module.execute to prevent possible silent memory corruption when too many EValue inputs are passed.

Fixes#10510

Test plan

Added unit test TestExecuteWithTooManyInputs
Verified by successfully running all module_test.cpp tests, except TestPTD (did not have access to ModuleLinear.ptd)
To run locally:
- Bypass is_fbcode guard in targets.bzl and redirect test file paths to use a locally exported ModuleAdd.pte file
- Build and run tests via:
```
buck2 build //extension/module/test:test
buck2 run //extension/module/test:test
```

pytorch-bot · 2025-05-05T23:56:29Z

🔗 Helpful Links

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

📄 Preview Python docs built from this PR

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

❌ 1 New Failure, 1 Unrelated Failure

As of commit bb83697 with merge base d069d65 ():

NEW FAILURE - The following job has failed:

pull / unittest-editable / linux / linux-job (gh)
extension/llm/modules/test/test_position_embeddings.py::TiledTokenPositionalEmbeddingTest::test_tiled_token_positional_embedding_et

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

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

pull / test-llava-runner-linux / linux-job (gh) (trunk failure)
test_llava_export

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

keyprocedure · 2025-05-05T23:58:55Z

@pytorchbot label "topic: not user facing"

keyprocedure · 2025-05-15T02:14:22Z

@pytorchbot label "release notes: none"

keyprocedure · 2025-05-19T19:05:59Z

Hi @shoumikhin, the CI failures are unrelated to this PR; they're due totest_llava_export and test_position_embeddings.py. Could you please merge when you get a chance?

### Summary Adds input size validation to `Module.execute` to prevent possible silent memory corruption when too many EValue inputs are passed. Fixespytorch#10510 ### Test plan - Added unit test `TestExecuteWithTooManyInputs` - Verified by successfully running all `module_test.cpp` tests, except `TestPTD` (did not have access to `ModuleLinear.ptd`) - To run locally: - Bypass `is_fbcode` guard in `targets.bzl` and redirect test file paths to use a locally exported `ModuleAdd.pte` file - Build and run tests via: ``` buck2 build //extension/module/test:test buck2 run //extension/module/test:test --------- Co-authored-by: Anthony Shoumikhin <anthony@shoumikh.in>

Add input size check and unit test
b743c13

keyprocedure requested a review from shoumikhin as a code owner May 5, 2025 23:56

facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 5, 2025

pytorch-botBot added the topic: not user facing label May 5, 2025

Apply lintrunner formatting
a7941df

pytorch-botBot added the release notes: none Do not include this in the release notes label May 15, 2025

Merge branch 'main' into add-module-execute-input-count-check
d7aa520

shoumikhin approved these changes May 16, 2025
View reviewed changes

Merge branch 'main' into add-module-execute-input-count-check
bb83697

shoumikhin merged commit e365e15 into pytorch:mainMay 19, 2025

Add input size check and unit test
b743c13

keyprocedure requested a review from shoumikhin as a code owner May 5, 2025 23:56

facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 5, 2025

pytorch-botBot added the topic: not user facing label May 5, 2025

Apply lintrunner formatting
a7941df

pytorch-botBot added the release notes: none Do not include this in the release notes label May 15, 2025

Merge branch 'main' into add-module-execute-input-count-check
d7aa520

shoumikhin approved these changes May 16, 2025
View reviewed changes

Merge branch 'main' into add-module-execute-input-count-check
bb83697

shoumikhin merged commit e365e15 into pytorch:mainMay 19, 2025

Add input size check and unit test
b743c13

keyprocedure requested a review from shoumikhin as a code owner May 5, 2025 23:56

facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 5, 2025

pytorch-botBot added the topic: not user facing label May 5, 2025

Apply lintrunner formatting
a7941df

pytorch-botBot added the release notes: none Do not include this in the release notes label May 15, 2025

Merge branch 'main' into add-module-execute-input-count-check
d7aa520

shoumikhin approved these changes May 16, 2025
View reviewed changes

Merge branch 'main' into add-module-execute-input-count-check
bb83697

shoumikhin merged commit e365e15 into pytorch:mainMay 19, 2025

Add input size check and unit test
b743c13

keyprocedure requested a review from shoumikhin as a code owner May 5, 2025 23:56

facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 5, 2025

pytorch-botBot added the topic: not user facing label May 5, 2025

Apply lintrunner formatting
a7941df

pytorch-botBot added the release notes: none Do not include this in the release notes label May 15, 2025

Merge branch 'main' into add-module-execute-input-count-check
d7aa520

shoumikhin approved these changes May 16, 2025
View reviewed changes

Merge branch 'main' into add-module-execute-input-count-check
bb83697

shoumikhin merged commit e365e15 into pytorch:mainMay 19, 2025

Add input size check and unit test
b743c13

keyprocedure requested a review from shoumikhin as a code owner May 5, 2025 23:56

facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 5, 2025

pytorch-botBot added the topic: not user facing label May 5, 2025

Apply lintrunner formatting
a7941df

pytorch-botBot added the release notes: none Do not include this in the release notes label May 15, 2025

Merge branch 'main' into add-module-execute-input-count-check
d7aa520

shoumikhin approved these changes May 16, 2025
View reviewed changes

Merge branch 'main' into add-module-execute-input-count-check
bb83697

shoumikhin merged commit e365e15 into pytorch:mainMay 19, 2025

Add input size check and unit test
b743c13

keyprocedure requested a review from shoumikhin as a code owner May 5, 2025 23:56

facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 5, 2025

pytorch-botBot added the topic: not user facing label May 5, 2025

Apply lintrunner formatting
a7941df

pytorch-botBot added the release notes: none Do not include this in the release notes label May 15, 2025

Merge branch 'main' into add-module-execute-input-count-check
d7aa520

shoumikhin approved these changes May 16, 2025
View reviewed changes

Merge branch 'main' into add-module-execute-input-count-check
bb83697

shoumikhin merged commit e365e15 into pytorch:mainMay 19, 2025

Add input size check and unit test
b743c13

keyprocedure requested a review from shoumikhin as a code owner May 5, 2025 23:56

facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 5, 2025

pytorch-botBot added the topic: not user facing label May 5, 2025

Apply lintrunner formatting
a7941df

pytorch-botBot added the release notes: none Do not include this in the release notes label May 15, 2025

Merge branch 'main' into add-module-execute-input-count-check
d7aa520

shoumikhin approved these changes May 16, 2025
View reviewed changes

Merge branch 'main' into add-module-execute-input-count-check
bb83697

shoumikhin merged commit e365e15 into pytorch:mainMay 19, 2025

Add input size check and unit test
b743c13

keyprocedure requested a review from shoumikhin as a code owner May 5, 2025 23:56

facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 5, 2025

pytorch-botBot added the topic: not user facing label May 5, 2025

Apply lintrunner formatting
a7941df

pytorch-botBot added the release notes: none Do not include this in the release notes label May 15, 2025

Merge branch 'main' into add-module-execute-input-count-check
d7aa520

shoumikhin approved these changes May 16, 2025
View reviewed changes

Merge branch 'main' into add-module-execute-input-count-check
bb83697

shoumikhin merged commit e365e15 into pytorch:mainMay 19, 2025

Uh oh!

Conversation

keyprocedure commented May 5, 2025

Summary

Test plan

Uh oh!

pytorch-botBot commented May 5, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🔗 Helpful Links

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

❌ 1 New Failure, 1 Unrelated Failure

Uh oh!

keyprocedure commented May 5, 2025

Uh oh!

keyprocedure commented May 15, 2025

Uh oh!

keyprocedure commented May 19, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

keyprocedure commented May 5, 2025

Summary

Test plan

Uh oh!

pytorch-botBot commented May 5, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🔗 Helpful Links

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

❌ 1 New Failure, 1 Unrelated Failure

Uh oh!

keyprocedure commented May 5, 2025

Uh oh!

keyprocedure commented May 15, 2025

Uh oh!

keyprocedure commented May 19, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

keyprocedure commented May 5, 2025

Summary

Test plan

Uh oh!

pytorch-botBot commented May 5, 2025• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🔗 Helpful Links

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

❌ 1 New Failure, 1 Unrelated Failure

Uh oh!

keyprocedure commented May 5, 2025

Uh oh!

keyprocedure commented May 15, 2025

Uh oh!

keyprocedure commented May 19, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

pytorch-botBot commented May 5, 2025•
edited
Loading

pytorch-botBot commented May 5, 2025•
edited
Loading

pytorch-botBot commented May 5, 2025•
edited
Loading

pytorch-botBot commented May 5, 2025•
edited
Loading

pytorch-botBot commented May 5, 2025•
edited
Loading