Skip to content
This repository has been archived by the owner on Dec 1, 2021. It is now read-only.

Update rbf and preloader #381

Merged
merged 3 commits into from Aug 15, 2019
Merged

Update rbf and preloader #381

merged 3 commits into from Aug 15, 2019

Conversation

n-nez
Copy link
Contributor

@n-nez n-nez commented Aug 8, 2019

Motivation and Context

150MHz version of IP

Description

How has this been tested?

Tested on board with IP standalone testing program.
Plus dlk CI supposed to be run on this PR.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature / Optimization (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@n-nez n-nez added enhancement New feature or request CI: auto-run Run CI automatically CI: test-dlk Run dlk test once labels Aug 8, 2019
@n-nez n-nez requested a review from iizukak August 8, 2019 10:07
@blueoil-butler blueoil-butler bot removed the CI: test-dlk Run dlk test once label Aug 8, 2019
@iizukak
Copy link
Member

iizukak commented Aug 9, 2019

@Joeper214 @ki-lm
Can you check this PR manually?
This branch can run inference on DE10-Nano.

Classification with LMNetV1Quantize is enough.

Copy link
Contributor

@Joeper214 Joeper214 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I verified it and it works on DE10-nano.

@iizukak
Copy link
Member

iizukak commented Aug 9, 2019

@Joeper214
Thanks 😄
I ask @ki-lm to verify on Object Detection.

@iizukak
Copy link
Member

iizukak commented Aug 9, 2019

run test

@blueoil-butler blueoil-butler bot added CI: test-all Run all tests once and removed CI: test-all Run all tests once labels Aug 9, 2019
@hadusam hadusam added the CI: test-all Run all tests once label Aug 13, 2019
@blueoil-butler blueoil-butler bot removed the CI: test-all Run all tests once label Aug 13, 2019
@iizukak iizukak merged commit a8f8f3c into blue-oil:master Aug 15, 2019
@Joeper214
Copy link
Contributor

Joeper214 commented Aug 15, 2019

@n-nez

Thanks. I confirmed the changes by running separate Measurements on DE-10 Nano.

I used Classification with LMNetV1Quantize.

Before change

-------------------------------------------------------------
Comparison: Default network test  succeeded!!!
-------------------------------------------------------------
TotalInitTime 14388,  sum:14.388ms
TotalRunTime 15992,  sum:15.992ms
..Convolution 11173,119,  sum:11.292ms
....kn2row 11039,  sum:11.039ms
......kn2row-buf 6,  sum:0.006ms
......matrix_multiplication 6731,  sum:6.731ms
........matrix_transpose (row_major) 42,  sum:0.042ms
......matrix_shift_add_f1 778,  sum:0.778ms
......matrix_shift_add_f2 3476,  sum:3.476ms
....kn2row-1x1 111,  sum:0.111ms
......matrix_multiplication 96,  sum:0.096ms
..BatchNorm 284,66,  sum:0.35ms
..QTZ_linear_mid_tread_half 1225,  sum:1.225ms
....pack_input 101,  sum:0.101ms
..QuantizedConv2D 698,1024,386,405,117,  sum:2.63ms
....Tensor convert 134,20,12,5,7,  sum:0.178ms
....Sync UDMABuf Input 86,80,47,31,26,  sum:0.27ms
....Conv2D TCA 390,862,275,327,42,  sum:1.896ms
....Sync UDMABuf Output 57,39,30,23,20,  sum:0.169ms
..Memcpy 59,25,15,8,  sum:0.107ms
..ExtractImagePatches 58,12,13,  sum:0.083ms
..QuantizedConv2D_ApplyScalingFactor 39,  sum:0.039ms
..ReLu 18,  sum:0.018ms
..Add 9,  sum:0.009ms
..AveragePool 13,  sum:0.013ms
..SoftMax 113,  sum:0.113ms

After change

Comparison: Default network test  succeeded!!!
-------------------------------------------------------------
TotalInitTime 13710,  sum:13.71ms
TotalRunTime 15756,  sum:15.756ms
..Convolution 11458,121,  sum:11.579ms
....kn2row 11331,  sum:11.331ms
......kn2row-buf 6,  sum:0.006ms
......matrix_multiplication 6932,  sum:6.932ms
........matrix_transpose (row_major) 43,  sum:0.043ms
......matrix_shift_add_f1 775,  sum:0.775ms
......matrix_shift_add_f2 3570,  sum:3.57ms
....kn2row-1x1 112,  sum:0.112ms
......matrix_multiplication 95,  sum:0.095ms
..BatchNorm 296,70,  sum:0.366ms
..QTZ_linear_mid_tread_half 1157,  sum:1.157ms
....pack_input 101,  sum:0.101ms
..QuantizedConv2D 630,774,319,326,111,  sum:2.16ms
....Tensor convert 135,22,12,5,7,  sum:0.181ms
....Sync UDMABuf Input 87,79,46,30,25,  sum:0.267ms
....Conv2D TCA 323,610,210,248,36,  sum:1.427ms
....Sync UDMABuf Output 56,39,30,23,21,  sum:0.169ms
..Memcpy 58,25,15,8,  sum:0.106ms
..ExtractImagePatches 58,12,13,  sum:0.083ms
..QuantizedConv2D_ApplyScalingFactor 38,  sum:0.038ms
..ReLu 18,  sum:0.018ms
..Add 9,  sum:0.009ms
..AveragePool 13,  sum:0.013ms
..SoftMax 116,  sum:0.116ms

@primenumber primenumber mentioned this pull request Aug 15, 2019
5 tasks
@iizukak iizukak added this to the v0.11.0 milestone Sep 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CI: auto-run Run CI automatically enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants