-
Notifications
You must be signed in to change notification settings - Fork 312
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.
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
Inspector should allow explicit None for delegate_map #4136
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/4136
Note: Links to docs will display an error until the docs builds have been completed. ❌ 1 New Failure, 1 Unrelated FailureAs of commit d9b0090 with merge base c839b9e (): NEW FAILURE - The following job has failed:
BROKEN TRUNK - The following job failed but were present on the merge base:👉 Rebase onto the `viable/strict` branch to avoid these failures
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This pull request was exported from Phabricator. Differential Revision: D59298227 |
This pull request was exported from Phabricator. Differential Revision: D59298227 |
Summary: Pull Request resolved: pytorch#4136 The SDK inspector should allow parsing delegates that have a `None` for their delegate_map. This is explicitly None, and not unspecified, which is why the default of an empty dict didn't work. Use `or {}` to instead detect None. Reviewed By: Olivia-liu Differential Revision: D59298227
2282441
to
6c5bc83
Compare
This pull request was exported from Phabricator. Differential Revision: D59298227 |
Summary: Pull Request resolved: pytorch#4136 The SDK inspector should allow parsing delegates that have a `None` for their delegate_map. This is explicitly None, and not unspecified, which is why the default of an empty dict didn't work. Use `or {}` to instead detect None. Reviewed By: Olivia-liu Differential Revision: D59298227
6c5bc83
to
5d78a7d
Compare
Summary: Pull Request resolved: pytorch#4136 The SDK inspector should allow parsing delegates that have a `None` for their delegate_map. This is explicitly None, and not unspecified, which is why the default of an empty dict didn't work. Use `or {}` to instead detect None. Reviewed By: Olivia-liu Differential Revision: D59298227
This pull request was exported from Phabricator. Differential Revision: D59298227 |
5d78a7d
to
d9b0090
Compare
This pull request has been merged in 970e278. |
Summary:
The SDK inspector should allow parsing delegates that have a
None
for their delegate_map. This is explicitly None, and not unspecified, which
is why the default of an empty dict didn't work.
Use
or {}
to instead detect None.Differential Revision: D59298227