wasi-nn: update the ort library#12162
Conversation
|
Can someone tell me how to fix CI to pass? |
|
The failing CI job is right to fail: it's indicating that pulling in the new library requires new vetting. One of the core team members will have to do a I'm tagged as reviewer here but I have some high-priority stuff on my plate at the moment so it may not be until sometime next week when I get to this, sorry! |
cfallin
left a comment
There was a problem hiding this comment.
Alright, I had some time to do the vets and the changes here look good. Thanks!
|
Understand. Thank you! |
|
@tamaroning I think this needs a Cargo.lock update as well (see the CI failures); would you mind doing that by doing |
|
Hi, I ran |
Head branch was pushed to by a user without write access
No, I didn't mean Please see the CI job failure here: that is showing that after running several |
|
Hi, I reset my local branch to 0da9d0b and ran all |
|
Hmm, I wasn't able to reproduce that either. I re-triggered the CI jobs to see if it was some transient version issue (e.g. maybe a release of |
|
OK, that didn't work, and the new failures don't seem reproducible either. I think at this point I need to ask our resident CI herder @alexcrichton (when he's back in office) for help... |
|
To reproduce the error you'll want to run a |
ca258b2 to
cbb5e0b
Compare
|
Thank you. I rebased the branch and could update Cargo.lock. |
FIx #12161