executorch icon indicating copy to clipboard operation
executorch copied to clipboard

Add mypy ignore for snakeviz

Open perheld opened this issue 1 month ago • 1 comments

Add a import-untyped for snakeviz in the case it is installed.

Signed-off-by: [email protected] Change-Id: Ia951a0013d09e06c0d29a32bdb6b49ae11561d7d

cc @freddan80 @per @zingo @oscarandersson8218 @digantdesai

perheld avatar Nov 13 '25 08:11 perheld

:link: Helpful Links

:test_tube: See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/15806

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

:x: 2 New Failures, 3 Unrelated Failures

As of commit 4ea0bce2756f437dbd749c14736b3adc294cc8c9 with merge base 65d4b941bafeb496a11786819ce2df7529aec512 (image):

NEW FAILURES - The following jobs have failed:

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.

pytorch-bot[bot] avatar Nov 13 '25 08:11 pytorch-bot[bot]

Hi @metascroy here is one PR outside or Arm stuff we might want help to review!

zingo avatar Nov 17 '25 10:11 zingo

@metascroy has imported this pull request. If you are a Meta employee, you can view this in D87489291.

meta-codesync[bot] avatar Nov 19 '25 23:11 meta-codesync[bot]

Rebased to make sure the cortex-m size test has been fixed. This PR should not in any way cause problems in it but its nice to prove green CI on all Arm tests.

zingo avatar Nov 20 '25 05:11 zingo

@metascroy I rebased this after your import and need a re-import, Thinking about about it the rebase was a bit stupid but now its done. Sorry for the extra work.

zingo avatar Nov 20 '25 05:11 zingo

Hi @metascroy here is one PR outside or Arm stuff we might want help to review!

The internal tests look good if you want to merge

metascroy avatar Nov 21 '25 17:11 metascroy