Arm backend: Bump vela to 5.1.0#20181
Conversation
Signed-off-by: Oscar Andersson <oscar.andersson@arm.com> Change-Id: I0f5d97e2eb3cbb2d9c2a74eecc6f2f9d26386a09
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/20181
Note: Links to docs will display an error until the docs builds have been completed. ❌ 1 New FailureAs of commit 39c2a4f with merge base c7a0b68 ( NEW FAILURE - The following job has failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
|
@digantdesai I think this one needs the usual magic @per also just merged a bump of Tosa tools |
|
@digantdesai OK to merge? |
relda88
left a comment
There was a problem hiding this comment.
The happy path is clean. What happens if the third-party API returns 429 here? Might be worth handling that case explicitly.
@relda88 I'm not following what you mean. Can you be a bit more specific? |
|
Thanks @zingo for flagging this. I need to update the internal version too. Also for TOSA tools. |
|
@rascani has imported this pull request. If you are a Meta employee, you can view this in D109491160. |
digantdesai
left a comment
There was a problem hiding this comment.
Review automatically exported from Phabricator review in Meta.
Bumps vela to 5.1.0 and removes xfails that been fixed.
cc @digantdesai @freddan80 @per @zingo @mansnils @Sebastian-Larsson @robell @rascani