Skip to content

Conversation

perheld
Copy link
Collaborator

@perheld perheld commented Feb 3, 2025

As suggested in a previous discussion the tosa serializer should go into .mypy.ini for a project wide ignore on missing imports instead of doing inline annotations in every file.

cc @digantdesai @freddan80 @per @zingo @oscarandersson8218

Copy link

pytorch-bot bot commented Feb 3, 2025

🔗 Helpful Links

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

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

✅ You can merge normally! (2 Unrelated Failures)

As of commit 5fd5a7e with merge base a5c7609 (image):

BROKEN TRUNK - The following jobs 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.

@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 Feb 3, 2025
@perheld perheld added topic: not user facing and removed CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. labels Feb 3, 2025
@perheld
Copy link
Collaborator Author

perheld commented Feb 3, 2025

@pytorchbot label ciflow/trunk

@perheld perheld added the partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm label Feb 3, 2025
@zingo zingo requested a review from mergennachin February 3, 2025 07:27
@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 Feb 3, 2025
As suggested in a previous discussion the tosa serializer should go
into .mypy.ini for a project wide ignore on missing imports instead of
doing inline annotations in every file.

Change-Id: I3ea00d0ff586d0f68a2c0fc557a2296cd03f3b04
@perheld perheld force-pushed the upstream/ph-mypy-ini-tosa-serialiser branch from 4d19888 to 5fd5a7e Compare February 3, 2025 09:24
@mergennachin mergennachin merged commit 38e0bc7 into pytorch:main Feb 3, 2025
106 of 108 checks passed
@perheld perheld deleted the upstream/ph-mypy-ini-tosa-serialiser branch February 7, 2025 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm topic: not user facing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants