Jovan Mitrevski

Actually, I see that #591 quite extensively modifies the code in this PR, so evaluating this PR independently I think is not useful. Extracting those changes is not very easy...

Thanks for this PR. It will be very useful for me. When I tried synthesizing the code that has many 1D CNNs with Vitis, I get the following error. ```...

Some warnings I get: ``` WARNING: [HLS 207-5551] unexpected pragma argument 'instances', expects function/operation (firmware/nnet_utils/nnet_conv1d_latency.h:67:24) WARNING: [HLS 207-5551] unexpected pragma argument 'instances', expects function/operation (firmware/nnet_utils/nnet_conv1d_latency.h:145:24) WARNING: [HLS 207-5556] Only for/while/do...

Are there still issues with the resource implementation? I am testing this on a 1D CNN, and the compiler seems to fail when I use latency. This particular has a...

Using the resource implementation, I have successfully used this backend for a 1D CNN Vitis workflow.

The FINN people mentioned the default pipeline time changing. Are there any issues related to that?

This has been tested for 1D, and could be merged in the more restricted sense, but it has not been tested for 2D. I think planning should take into account...

I will restrict this PR to the 1D case because I think that's the only one that would be important. If we fix it for 2D, we can make a...

Can you provide more information on how to reproduce the problem?

InplaceVariables can get messed up by optimizer passes. I fixed the issue in a branch I am working on, but I can try to extract it and make a pull...