nntrainer icon indicating copy to clipboard operation
nntrainer copied to clipboard

[TF Exporter] FullyConnected Layer Weights Transpose for TFLite Export @open sesame 10/21 18:26

Open DonghakPark opened this issue 2 years ago • 10 comments

This PR for TFlite export Transpose FullyConnected Layer's weights (NCHW --> NHWC)

in shape 1:3:2:4 & FC with unit 1

Before reordering 0 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

After reordering 0 8 16 1 9 17 2 10 18 3 11 19 4 12 20 5 13 21 6 14 22 7 15 23

The Result of input - flatten - FC predicted as sum(Wi * Xi) + bias ==> 4325 and the ouput also 4325

Related : #1912 Close : #1973 Signed-off-by: DonghakPark [email protected]

DonghakPark avatar Sep 02 '22 08:09 DonghakPark

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1994. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

taos-ci avatar Sep 02 '22 08:09 taos-ci

:octocat: cibot: @DonghakPark, The last line of a text file must have a newline character. Please append a new line at the end of the line in test/unittest/compiler/unittest_interpreter.cpp.

taos-ci avatar Sep 02 '22 08:09 taos-ci

:octocat: cibot: @DonghakPark, The last line of a text file must have a newline character. Please append a new line at the end of the line in test/unittest/compiler/unittest_interpreter.cpp.

taos-ci avatar Sep 02 '22 08:09 taos-ci

:octocat: cibot: @DonghakPark, The last line of a text file must have a newline character. Please append a new line at the end of the line in test/unittest/compiler/unittest_interpreter.cpp.

taos-ci avatar Sep 02 '22 09:09 taos-ci

:octocat: cibot: @DonghakPark, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1994-202209021801520.22432708740234-bd7ae2c5ecff0ff05120ba67466942cf524fe7eb/.

taos-ci avatar Sep 02 '22 09:09 taos-ci

To contributor, We have used 'Signed-off-by:' notation by default to handle the license issues, that result from contributors. Note that 'Is there a Signed-off-by line?' is important because lawyers tell us we must have to it to cleanly maintain the open-source license issues even though it has nothing to do with the code itself.

taos-ci avatar Sep 14 '22 00:09 taos-ci

I think commit title also need to change. when this commit is merged.. it is not draft any more.

jijoongmoon avatar Sep 14 '22 05:09 jijoongmoon

I think commit title also need to change. when this commit is merged.. it is not draft any more.

Fixed (Title and Comment)

DonghakPark avatar Sep 14 '22 06:09 DonghakPark

:octocat: cibot: @DonghakPark, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1994-202209301557340.74472904205322-d88c0c310f3bbd73ac63ecd01403afa1c3858df6/.

taos-ci avatar Sep 30 '22 07:09 taos-ci

The Ubuntu error might be related with #2013

myungjoo avatar Oct 04 '22 13:10 myungjoo