Skip to content
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

[IENN]Integrate tranpose_sinking pass #62

Open
wants to merge 1 commit into
base: webml
Choose a base branch
from

Conversation

lisa0314
Copy link

@lisa0314 lisa0314 commented Nov 6, 2020

Model No tranpose After insert  transpose After integrate transpose_sinking
Deeplab 224(tf) 13.46+-0.29ms 23.49+-1.12ms 13.91+-0.36ms
Deeplab 257(tf) 17.16+-0.48ms 32.67+-1.74ms 18.13+-0.63ms
Deeplab 321(tf) 25.31+-0.48ms 50.94+-1.98ms 26.94+-0.75ms
Deeplab 513(tf) 82.78+-24.35ms 190.09+-34.39ms 84.84+-26.52ms
  IE-MKLDNN IE-clDNN
baseline pass: 604 fail: 294 pass: 604 fail: 294
current pass: 604 fail: 294 pass: 604 fail: 294

@lisa0314
Copy link
Author

lisa0314 commented Nov 16, 2020

@fujunwei Please help to review. Thanks!

@fujunwei
Copy link

Please update binary for windows

fujunwei pushed a commit that referenced this pull request Apr 22, 2022
There are 2 problems being fixed:
(1) typeSwapBookmarksIfNecessary now stores the type-swapped items
    in a different array. This is the array to search for the new
    bookmark.
(2) sSkipShowSaveFlowForTesting can be null and the if statement
    doesn't account for that.

(cherry picked from commit 237a880)

Bug: 1315511
Change-Id: Iac5c43c847085e4b64e4453540c4c4d6492126ee
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3580056
Reviewed-by: bttk - <bttk@chromium.org>
Commit-Queue: Brandon Wylie <wylieb@chromium.org>
Cr-Original-Commit-Position: refs/heads/main@{#992820}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3594486
Reviewed-by: Krishna Govind <govind@chromium.org>
Commit-Queue: Krishna Govind <govind@chromium.org>
Owners-Override: Krishna Govind <govind@chromium.org>
Cr-Commit-Position: refs/branch-heads/5005@{#62}
Cr-Branched-From: 5b4d945-refs/heads/main@{#992738}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants