fastmachinelearning / hls4ml

Machine learning on FPGAs using HLS
https://fastmachinelearning.org/hls4ml
Apache License 2.0
1.24k stars 402 forks source link

Explicitly set strategy for pointwise conv #785

Closed vloncar closed 1 year ago

vloncar commented 1 year ago

Description

When optimizing 1x1 convolution a new node will be inserted with the same name and the user config will be used. This means the new PointwiseConv1D/2D node will pick up whatever string the user set in the config and unless that matches lowercase latency or resource the compilation will fail. This was observed in #782.

The solution is not elegant, but it was the least intrusive. It should be fine until we rework the config itself.

Type of change

Tests

The test_pointwiseconv.py was extended to incude a test for this scenario.

Checklist