Closed jmduarte closed 8 months ago
pre-commit.ci autofix
@jmduarte Can you re-base this to the lastest changes from #815? I'd love to try this out after that!
@jmduarte I'm actually trying this out now, but I just realized it is in vivado, is it possible to update this to vitis?? I would be happy to contribute if you want!!
Superseded by #881
Description
This is mostly for discussion and to let others test it out like @Duchstf. This PR adds an explicit pointwise Conv1D implementation, where the reuse factor (
RF
) is used to split the layer execution and reuse the existing moduleRF
timesOriginal pointwise Conv1D:
(in_width, n_chan) -> (in_width, n_filt)
This PR splits it into
RF
calls of(in_width/RF, n_chan) -> (in_width/RF, n_filt)
(in_width/RF, n_chan) -> (in_width/RF, n_filt)
(in_width/RF, n_chan) -> (in_width/RF, n_filt)
The II ~ RF. To turn it on you have to configure
ConvImplementation
of the layer named<layer>
Limitations:
in_width
is divisible byRF
RF = 120
. Could be automated with code generation.Type of change
Tests
See test/pytest/test_pointwiseconv.py
Checklist
pre-commit
on the files I edited or added.