-
Notifications
You must be signed in to change notification settings - Fork 3k
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
turn on neural_speed by default #19627
Conversation
In my opinion the problem still should be fixed, especially we see the fix is there but for what reason we refuse doing so? Windows and Office are deployed to billions of devices. So definitely some people will hit the issue. Even if they only represent a small percentage of the total population, we will still see a huge number of crash reports. |
the fix is in progress. At the same time, we don't want user to experience very slow experience for the corner case. |
04d63bf
to
0f5cf75
Compare
0f5cf75
to
f12a66d
Compare
### Description <!-- Describe your changes. --> the crash caused by the neural_speed turns out to be a very corn case. Turn it on by default. ### Motivation and Context <!-- - Why is this change required? What problem does it solve? - If it fixes an open issue, please link to the issue here. -->
Description
the crash caused by the neural_speed turns out to be a very corn case. Turn it on by default.
Motivation and Context