-
Notifications
You must be signed in to change notification settings - Fork 576
Add README.md for the NXP eIQ Neutron Backend #9718
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add README.md for the NXP eIQ Neutron Backend #9718
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/9718
Note: Links to docs will display an error until the docs builds have been completed. ❌ 54 New Failures, 7 Pending, 12 Unrelated FailuresAs of commit 95f4134 with merge base 771588a ( NEW FAILURES - The following jobs have failed:
FLAKY - The following jobs failed but were likely due to flakiness present on trunk:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
Hi @robert-kalmar! Thank you for your pull request and welcome to our community. Action RequiredIn order to merge any pull request (code, docs, etc.), we require contributors to sign our Contributor License Agreement, and we don't seem to have one on file for you. ProcessIn order for us to review and merge your suggested changes, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA. Once the CLA is signed, our tooling will perform checks and validations. Afterwards, the pull request will be tagged with If you have received this in error or have any questions, please contact us at [email protected]. Thanks! |
@robert-kalmar once you sign the CLA, I can add you to the list where you can automatically run CI and edit labels etc. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @robert-kalmar and the NXP team for the very first ET PR! Looking forward to a productive collaboration!! PR looks good. Left a couple of nits.
eIQ extensions connect broader AI ecosystems to the edge, such as the NVIDIA TAO extension, which enables developers to bring AI models trained and fine-tuned with TAO to NXP-powered edge devices. | ||
|
||
|
||
## Supported NXP platforms |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a list of NPU supported i.e. Neutron N3 IIRC?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, that is good idea to list the Neutron variants too.
|
||
## Help & Improvements | ||
If you have problems or questions or have suggestions for ways to make | ||
implementation and testing better, please reach out to the NXP representative for the SoC you are interested in using, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Request for creating a github issue with modules: nxp
label instead?
Created modules: nxp
label for this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for introducing the label. The Help & improvements section is updated.
## Layout | ||
TBD | ||
|
||
## Backend Status and Maturity |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Backend Status and Maturity
Current Status: Prototype Quality
The eIQ Neutron NPU Backend should be considered as prototype quality at this moment...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is easier to see.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree, updated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @robert-kalmar
Yey CLA is green now! @robert-kalmar do you want to fix the nits? And push again, I will merge it then. |
Yes, the CLA is resolved! I will fix the nits and update the PR. |
69dcbf4
to
95f4134
Compare
@digantdesai , @mergennachin, the PR is updated with your suggestions. Thank you for them. If you are OK, you can merge this PR. |
CI is unhappy but seems unrelated. Let's go! |
### Summary Initial Readme, introducing the NXP's eIQ Neutron Backend, supporting the eIQ Neutron NPU hardware accelerator.
Summary
Initial Readme, introducing the NXP's eIQ Neutron Backend, supporting the eIQ Neutron NPU hardware accelerator.