Skip to content

NXP backend: handle tests of mlperf tiny visual wake words - #22844

Merged
novak-vaclav merged 1 commit into
pytorch:mainfrom
nxp-upstream:feature/EIEX-1031-refactor-mlperf-tiny-vww
Sep 22, 2026
Merged

novak-vaclav merged 1 commit into
pytorch:mainfrom
nxp-upstream:feature/EIEX-1031-refactor-mlperf-tiny-vww

Conversation

@novak-vaclav

Copy link
Copy Markdown
Collaborator

Summary

Enable and refactor MLPerf Tiny Visual Wake Words tests.

Test plan

tests can be manually run using pytest -c /dev/null backends/nxp/tests/

cc @robert-kalmar @JakeStevens @digantdesai @rascani @roman-janik-nxp

Copilot AI lite review requested due to automatic review settings September 15, 2026 16:28
@novak-vaclav novak-vaclav added module: nxp Issues related to NXP Neutron NPU delegation and code under backends/nxp/ release notes: nxp Changes to the NXP Neutron backend delegate labels Sep 15, 2026
@pytorch-bot

pytorch-bot Bot commented Sep 15, 2026 •

Copy link
Copy Markdown

🔗 Helpful Links

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

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

✅ No Failures

As of commit 8539ff4 with merge base 3ffd297 (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 15, 2026
@linux-foundation-easycla

linux-foundation-easycla Bot commented Sep 15, 2026 •

Copy link
Copy Markdown

CLA Not Signed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@novak-vaclav
novak-vaclav force-pushed the feature/EIEX-1031-refactor-mlperf-tiny-vww branch from b5c4aef to 8fc8755 Compare September 16, 2026 09:31
Copilot AI review requested due to automatic review settings September 16, 2026 09:31

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@novak-vaclav
novak-vaclav force-pushed the feature/EIEX-1031-refactor-mlperf-tiny-vww branch from 8fc8755 to ef87925 Compare September 16, 2026 09:32
@novak-vaclav

Copy link
Copy Markdown
Collaborator Author

Fixed invalid bound.

@roman-janik-nxp roman-janik-nxp left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No adding the model to aot_neutron_compile.py and tests for it?

Comment thread backends/nxp/tests/models/test_mlperf_tiny_visual_wake_words.py
@novak-vaclav
novak-vaclav force-pushed the feature/EIEX-1031-refactor-mlperf-tiny-vww branch from ef87925 to 4b4dac8 Compare September 16, 2026 15:39
Copilot AI review requested due to automatic review settings September 16, 2026 15:39

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@novak-vaclav

Copy link
Copy Markdown
Collaborator Author

You are right @roman-janik-nxp, I totally forgot about the aot_neutron_compile and its test. I added it now. Thank you.
Now it is ready for review.

@roman-janik-nxp roman-janik-nxp left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GJ!

@novak-vaclav
novak-vaclav force-pushed the feature/EIEX-1031-refactor-mlperf-tiny-vww branch from 4b4dac8 to 8539ff4 Compare September 21, 2026 13:56
Copilot AI review requested due to automatic review settings September 21, 2026 13:56

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@novak-vaclav

Copy link
Copy Markdown
Collaborator Author

Failing tests are not related to this code and internal tests are passing.
Merging.

@novak-vaclav
novak-vaclav merged commit 2bfe9fb into pytorch:main Sep 22, 2026
230 of 231 checks passed
@novak-vaclav
novak-vaclav deleted the feature/EIEX-1031-refactor-mlperf-tiny-vww branch September 22, 2026 14:09
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. module: nxp Issues related to NXP Neutron NPU delegation and code under backends/nxp/ release notes: nxp Changes to the NXP Neutron backend delegate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants