Skip to content

Conversation

kimishpatel
Copy link
Contributor

Reverts #13942

Broke a buck target test_neutoron_backend because of "ModuleNotFoundError: No module named 'executorch.backends.nxp._passes". _passes needs a buck target specified to be included as dep

@pytorch-bot pytorch-bot bot added the ci-no-td label Sep 22, 2025
Copy link

pytorch-bot bot commented Sep 22, 2025

🔗 Helpful Links

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

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

⏳ No Failures, 16 Pending

As of commit 5fd4ca8 with merge base 02d8710 (image):
💚 Looks good so far! There are no failures yet. 💚

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

@meta-cla meta-cla 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 Sep 22, 2025
Copy link

This PR needs a release notes: label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

@kimishpatel kimishpatel merged commit 99e4fbe into main Sep 22, 2025
131 of 134 checks passed
@kimishpatel kimishpatel deleted the revert-13942-upstream/main-nxp/EIEX-392-resolve-limitations-of-uncertain-tensor-formats branch September 22, 2025 19:15
StrycekSimon pushed a commit to nxp-upstream/executorch that referenced this pull request Sep 23, 2025
…pytorch#14475)

Reverts pytorch#13942

Broke a buck target test_neutoron_backend because of
"ModuleNotFoundError: No module named 'executorch.backends.nxp._passes".
_passes needs a buck target specified to be included as dep
@MartinPavella
Copy link
Collaborator

MartinPavella commented Sep 24, 2025

Hi @kimishpatel ,
how did you discover the issue? I couldn't find any relevant failing builds.
Could you please instruct me on how I can reproduce the failure?

@kimishpatel
Copy link
Contributor Author

@MartinPavella it was one of the internal builds that was failing. Just resubmit it and i will import to internal and make sure the failure is not there.

In this specific case it is actually whats in the summary "ModuleNotFoundError: No module named 'executorch.backends.nxp._passes". You need to create a buck target and add that as dep here

name = "test_neutron_backend",
, or include transitively in one of the other deps

@MartinPavella
Copy link
Collaborator

@MartinPavella it was one of the internal builds that was failing. Just resubmit it and i will import to internal and make sure the failure is not there.

In this specific case it is actually whats in the summary "ModuleNotFoundError: No module named 'executorch.backends.nxp._passes". You need to create a buck target and add that as dep here

name = "test_neutron_backend",

, or include transitively in one of the other deps

@kimishpatel Thank you for the reply.
I have implemented the changes in a new PR: #14576
Could you please check if your internal build works now?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-no-td CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants