-
Notifications
You must be signed in to change notification settings - Fork 21
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
Use opset15 version of Str Pack/Unpack #351
base: master
Are you sure you want to change the base?
Use opset15 version of Str Pack/Unpack #351
Conversation
49223d4
to
c7bb795
Compare
<td >4544</td> | ||
</tr> | ||
<tr> | ||
<td >SentencePiece</td> | ||
<td >89.19</td> | ||
<td >86.36</td> |
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.
why usage of official operations decreases pass rate?
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.
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.
Created CVS-159636 ticket for that
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.
I suppose we should wait for this bug to be fixed? before merging this PR
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.
I don't see much harm if we merge it how, but i'm not sure. @apaniukov how do you think?
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.
Maybe it is better to wait, but not for long.
c7bb795
to
06f4a20
Compare
e792446
to
5cb3efb
Compare
Ticket: CVS-155958