-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Conversation
Signed-off-by: Serge Panev <[email protected]>
* Move block.optimize_for backend_opts to kwargs Signed-off-by: Serge Panev <[email protected]> * Update Hybridize to use kwargs as backend opts Signed-off-by: Serge Panev <[email protected]> * Fix lint Signed-off-by: Serge Panev <[email protected]> * Change clear default to False and allow hybrize+optimize_for calls Signed-off-by: Serge Panev <[email protected]> * Fix nit Signed-off-by: Serge Panev <[email protected]> * Adress review comments Signed-off-by: Serge Panev <[email protected]> * Adress more review comments Signed-off-by: Serge Panev <[email protected]> * Adress more more review comments Signed-off-by: Serge Panev <[email protected]> * Fix nit Signed-off-by: Serge Panev <[email protected]>
…9652) Signed-off-by: Serge Panev <[email protected]>
Hey @Kh4L , Thanks for submitting the PR
CI supported jobs: [edge, centos-gpu, unix-cpu, windows-gpu, windows-cpu, miscellaneous, unix-gpu, website, clang, sanity, centos-cpu] Note: |
@mxnet-bot run ci [unix-gpu, centos-cpu] |
Jenkins CI successfully triggered : [unix-gpu, centos-cpu] |
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
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
Backported PRs