DigitalMediaProfessionals / tool

DV network conversion tool
Apache License 2.0
0 stars 1 forks source link

Added 1D layers support for Keras #23

Closed ZongHong-Lyu closed 6 years ago

ZongHong-Lyu commented 6 years ago

Caffe has not explicit 1D layers so only Keras needs to be updated.

This implementation treats those 1D layers as 2D layers with height = 1.

Also I fixed the split_pool_node function so now it can handle asymmetric kernel sizes, since 1D layers are like 2D layers with asymmetric kernels.

And in the fix, I also fixed a bug of pool strides of the split pool node.

This should also closes #16.

nhatuan84 commented 6 years ago

Hi Lyu,

I am testing types of layer related to Cov1D. And there are problems:

nhatuan84 commented 6 years ago

Hi Lyu,

I attached the test code, model here. I used PyWrapper:

https://dmprofs-my.sharepoint.com/:u:/g/personal/tuan_nhanh_dmprof_com/EYmUGjtBgvBLl5jHOOub4eMBQzuqm0D3EeXesgRlPNTipg?e=ffFY3P

ZongHong-Lyu commented 6 years ago

Hi Tuan,

I've tried but can't reproduce the error for the MaxPooling1D one. I'll report the issue of AveragePooling1D to the HW guy.

EDIT: Alexey also tested it and it seems pooling is not working using non-square configs.

ZongHong-Lyu commented 6 years ago

Hi Tuan,

The HW guy says the FPGA can't support 1D average pooling currently, He is working on fix that issue.

nhatuan84 commented 6 years ago

Hi Lyu,

Thank you.

But how about MaxPooling1D. Is your output similar to Keras 's output?

ZongHong-Lyu commented 6 years ago

Hi Tuan,

It seems work for the one test I did. But as I said above, Alexey made more tests and it seems max pooling is also not working.

nhatuan84 commented 6 years ago

Hi Lyu,

Thanks. Hope it will work soon since I have to deploy speech app to board soon.

ZongHong-Lyu commented 6 years ago

Hi Tuan,

We have uploaded new FPGA bit files to the DMP FTP. in /Jupiter/JupiterX/20181107 There are two files: boot.bin and boot.bin.SD. boot.bin is for eMMC and boot.bin.SD is for SD card (If you use SD card, rename it to boot.bin)

Can you try that one?

nhatuan84 commented 6 years ago

Hi Lyu,

i will test it. Thanks.

nhatuan84 commented 6 years ago

Hi Lyu,

It seems the HW was fixed.

Thanks.