Skip to content

Update Neutron SW to 3.1.3#20616

Merged
MartinPavella merged 1 commit into
pytorch:mainfrom
nxp-upstream:nxg01483/EIEX-983-update-eiq-neutron-sdk-to-3.1.3
Jul 2, 2026
Merged

Update Neutron SW to 3.1.3#20616
MartinPavella merged 1 commit into
pytorch:mainfrom
nxp-upstream:nxg01483/EIEX-983-update-eiq-neutron-sdk-to-3.1.3

Conversation

@MartinPavella

@MartinPavella MartinPavella commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

Summary

Update Neutron SW to the latest version (3.1.3) and update related tests accordingly.

Test plan

Tested by existing NXP backend tests.

cc @robert-kalmar @JakeStevens @digantdesai @rascani

@MartinPavella MartinPavella self-assigned this Jun 30, 2026
@MartinPavella MartinPavella 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 Jun 30, 2026
@pytorch-bot

pytorch-bot Bot commented Jun 30, 2026

Copy link
Copy Markdown

🔗 Helpful Links

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

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

⏳ No Failures, 10 Pending

As of commit f1f2ddf with merge base 2ec218c (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 Jun 30, 2026
@linux-foundation-easycla

linux-foundation-easycla Bot commented Jun 30, 2026

Copy link
Copy Markdown

CLA Not Signed

Comment thread docs/source/backends/nxp/nxp-overview.md Outdated
@MartinPavella MartinPavella force-pushed the nxg01483/EIEX-983-update-eiq-neutron-sdk-to-3.1.3 branch from 084b4aa to 132e43a Compare June 30, 2026 10:57
@MartinPavella MartinPavella requested a review from jirioc June 30, 2026 10:58
@MartinPavella MartinPavella marked this pull request as ready for review June 30, 2026 14:46
@robert-kalmar

Copy link
Copy Markdown
Collaborator

@MartinPavella temporarily increasing the time limit is OK, but please file a ticket internally to divide our tests into 2 parts - short running and long running.

The PR unit test executes all the time, multiple times when PR is raised. So in next sprint lets address this, to move the long running tests from pull to trunk workflow. So it will run only after PR is merged.

@MartinPavella MartinPavella force-pushed the nxg01483/EIEX-983-update-eiq-neutron-sdk-to-3.1.3 branch from e8aaf44 to 132e43a Compare July 1, 2026 11:26
@MartinPavella

Copy link
Copy Markdown
Collaborator Author

@MartinPavella temporarily increasing the time limit is OK, but please file a ticket internally to divide our tests into 2 parts - short running and long running.

The PR unit test executes all the time, multiple times when PR is raised. So in next sprint lets address this, to move the long running tests from pull to trunk workflow. So it will run only after PR is merged.

@robert-kalmar I'm sorry, I misread the time stamps. It is actually not necessary to increase the time limit. It appears that some test is hanging on the GitHub CI. The internal CI is passing, and all test pass locally too. On a recent standup you were discussing some issue with the mlperf tiny repo being cloned by the parallel workers, which caused some issues. Could that be the problem here? I assume the problem was discovered when I was on holiday.

@robert-kalmar

Copy link
Copy Markdown
Collaborator

@MartinPavella temporarily increasing the time limit is OK, but please file a ticket internally to divide our tests into 2 parts - short running and long running.
The PR unit test executes all the time, multiple times when PR is raised. So in next sprint lets address this, to move the long running tests from pull to trunk workflow. So it will run only after PR is merged.

@robert-kalmar I'm sorry, I misread the time stamps. It is actually not necessary to increase the time limit. It appears that some test is hanging on the GitHub CI. The internal CI is passing, and all test pass locally too. On a recent standup you were discussing some issue with the mlperf tiny repo being cloned by the parallel workers, which caused some issues. Could that be the problem here? I assume the problem was discovered when I was on holiday.

  • Hanging.. So it it not caused by introducing the the tests for big convolutions, like test__depthwise__big. Make sense as those tests were xfailed not skipped, so have run even before.
  • Definitelly not with the mlperf tiny repo, as it is not accessible outside, so no test clones it. But in general might be a problem with concurent access. We addressed where the tests write their artefacts. Maybe we have artefacts or resources we missed.

@MartinPavella MartinPavella force-pushed the nxg01483/EIEX-983-update-eiq-neutron-sdk-to-3.1.3 branch from b2e4e8f to f1f2ddf Compare July 2, 2026 06:35
@MartinPavella

Copy link
Copy Markdown
Collaborator Author

The skipped checks seem unrelated and the internal CI is passing. Merging.

@MartinPavella MartinPavella merged commit 3b0f1de into pytorch:main Jul 2, 2026
191 of 192 checks passed
@MartinPavella MartinPavella deleted the nxg01483/EIEX-983-update-eiq-neutron-sdk-to-3.1.3 branch July 2, 2026 08:08
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