search

DigitalMediaProfessionals / tool

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

Fixed parsing for built-in softmax layer for keras, added dilation at… #35

Closed ajkxyz closed 5 years ago

ajkxyz commented 5 years ago

…tribute to NodeParam, extended error message

ZongHong-Lyu commented 5 years ago

Hi Alexey,

Now I found that you also need to change fpga_layer.py so the dilation info is also output to generated c++ source code. Currently it output 0

ZongHong-Lyu commented 5 years ago

Hi Alexey,

After more close look, dilation field is added but not yet parsed. So maybe you mean to support dilation in a new pull request?

ajkxyz commented 5 years ago

Hi Lyu,

Yes, I've just added dilation field, as I needed it in performance estimation tool, the parsing of actual dilation should be implemented separately.

On Tue, Nov 27, 2018 at 2:36 PM ZongHong-Lyu notifications@github.com wrote:

Hi Alexey,

After more close look, dilation field is added but not yet parsed. So maybe you mean to support dilation in a new pull request?

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/DigitalMediaProfessionals/tool/pull/35#issuecomment-441925668, or mute the thread https://github.com/notifications/unsubscribe-auth/AGISDAtglQMeQltbz0mh4PQ2jOe4rOIDks5uzM9RgaJpZM4Yyf-9 .