Skip to content

Add support for method level executorch backend config#3266

Closed
tarun292 wants to merge 1 commit intomainfrom
export-D56499598
Closed

Add support for method level executorch backend config#3266
tarun292 wants to merge 1 commit intomainfrom
export-D56499598

Conversation

@tarun292
Copy link
Contributor

Summary: There are use cases where we might like to supply a separate ExecutorchBackendConfig for each method in the model. An example use case is where we might want to alloc inputs for one method and not alloc them for another. In order to support this, in this diff we add support for passing in a dictionary of configs to to_executorch.

Reviewed By: cccclai

Differential Revision: D56499598

@pytorch-bot
Copy link

pytorch-bot bot commented Apr 24, 2024

🔗 Helpful Links

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

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

✅ No Failures

As of commit d6c1a3d with merge base ebdb152 (image):
💚 Looks good so far! There are no failures yet. 💚

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

@facebook-github-bot 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 Apr 24, 2024
@facebook-github-bot
Copy link
Contributor

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

tarun292 added a commit that referenced this pull request Apr 24, 2024
Summary:

There are use cases where we might like to supply a separate ExecutorchBackendConfig for each method in the model. An example use case is where we might want to alloc inputs for one method and not alloc them for another. In order to support this, in this diff we add support for passing in a dictionary of configs to `to_executorch`.

Reviewed By: cccclai

Differential Revision: D56499598
@facebook-github-bot
Copy link
Contributor

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

facebook-github-bot pushed a commit that referenced this pull request May 3, 2024
Summary:

There are use cases where we might like to supply a separate ExecutorchBackendConfig for each method in the model. An example use case is where we might want to alloc inputs for one method and not alloc them for another. In order to support this, in this diff we add support for passing in a dictionary of configs to `to_executorch`.

Reviewed By: cccclai

Differential Revision: D56499598
@facebook-github-bot
Copy link
Contributor

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

tarun292 added a commit that referenced this pull request May 3, 2024
Summary:

There are use cases where we might like to supply a separate ExecutorchBackendConfig for each method in the model. An example use case is where we might want to alloc inputs for one method and not alloc them for another. In order to support this, in this diff we add support for passing in a dictionary of configs to `to_executorch`.

Reviewed By: cccclai

Differential Revision: D56499598
@tarun292 tarun292 force-pushed the export-D56499598 branch from 91533f5 to 5f860ca Compare May 3, 2024 08:14
@facebook-github-bot
Copy link
Contributor

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

facebook-github-bot pushed a commit that referenced this pull request May 9, 2024
Summary:

There are use cases where we might like to supply a separate ExecutorchBackendConfig for each method in the model. An example use case is where we might want to alloc inputs for one method and not alloc them for another. In order to support this, in this diff we add support for passing in a dictionary of configs to `to_executorch`.

Reviewed By: JacobSzwejbka, cccclai

Differential Revision: D56499598
@facebook-github-bot
Copy link
Contributor

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

tarun292 added a commit that referenced this pull request May 9, 2024
Summary:

There are use cases where we might like to supply a separate ExecutorchBackendConfig for each method in the model. An example use case is where we might want to alloc inputs for one method and not alloc them for another. In order to support this, in this diff we add support for passing in a dictionary of configs to `to_executorch`.

Reviewed By: JacobSzwejbka, cccclai

Differential Revision: D56499598
@tarun292 tarun292 force-pushed the export-D56499598 branch from 66cb1f4 to 6ffccc7 Compare May 9, 2024 20:34
@facebook-github-bot
Copy link
Contributor

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

facebook-github-bot pushed a commit that referenced this pull request May 9, 2024
Summary:

There are use cases where we might like to supply a separate ExecutorchBackendConfig for each method in the model. An example use case is where we might want to alloc inputs for one method and not alloc them for another. In order to support this, in this diff we add support for passing in a dictionary of configs to `to_executorch`.

Reviewed By: JacobSzwejbka, cccclai

Differential Revision: D56499598
@facebook-github-bot
Copy link
Contributor

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

Summary:

There are use cases where we might like to supply a separate ExecutorchBackendConfig for each method in the model. An example use case is where we might want to alloc inputs for one method and not alloc them for another. In order to support this, in this diff we add support for passing in a dictionary of configs to `to_executorch`.

Reviewed By: JacobSzwejbka, cccclai

Differential Revision: D56499598
@tarun292 tarun292 force-pushed the export-D56499598 branch from 2a38aab to d6c1a3d Compare May 9, 2024 20:46
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 2ac7f2a.

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

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants