-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
[microTVM][Zephyr] Add skip for AOT test #8628
Conversation
@mehrdadh Thanks a lot for improving the AOT test suite by skipping boards like ST Disco board, where the test in question doesn't fit! Also thanks for making it clear that the error number is a hex number. Small but important changes! :) The CI has failed, but I've got the same error in "BUILD: CPU" for my changes in PR 8613, which is a pure textual change, so I believe it's a CI glitch again (re-trigger so?). Bummer. The change LGTM! |
thanks for the review @gromero! I just triggered it. |
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.
This should really be two independent patches but otherwise LGTM
* add hex indicator to message * add pytest skip * trigger * trigger
* add hex indicator to message * add pytest skip * trigger * trigger
* add hex indicator to message * add pytest skip * trigger * trigger
* add hex indicator to message * add pytest skip * trigger * trigger
This PR adds
pytest.skip
for platforms that cannot fit the model.