-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Conversation
Thanks for your contribution! unix-gpu CI raises the following exception: C++ exception with description "[07:12:07] /work/mxnet/src/resource.cc:429: Check failed: state_space->ctx.dev_id == stream->dev_id (0 vs. -1) The device id of cudnn dropout state space doesn't match that from stream. Maybe we should modify the code of |
Thanks for your contribution! |
@mxnet-label-bot add [pr-awaiting-review] |
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.
Thanks for the fix
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.
LGTM.
Could you please retrigger the CI? Thanks!
seems empty commit did not trigger CI, trying close and reopen |
@roywei It seems that the CI has some problems, and all PRs are waiting for status to be reported. |
@roywei Hi. The problem of CI has been addressed. Could you please retrigger it? Thanks! |
Merged. Thanks! |
* use cudnn for dropout by default * update test * use dev_id * trigger ci * trigger ci
* use cudnn for dropout by default * update test * use dev_id * trigger ci * trigger ci
Description
enable cuddn for dropout after #13896
This was turned on only in Gluon dropout layers, but not in mx.nd.Dropout.
Test:
After setting changing the flag, ran the sample code in #13896, was able to get the same speed reported.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.
Changes
set cudnn_off default as false