hls4ml Optimization API [Part 2]

[bo3z]

Description

• Second part of hls4ml Optimization API hls4ml Optimization API [Part 1] #768
• Introduces Dense Unrolled layers, optimising multiplications with zero in Resource strategy with RF > 1
• Introduces additional TCL scripts, to optimise zero BRAM blocks.

Type of change

• New feature (non-breaking change which adds functionality)
• A new research paper code implementation
• Fix issue Vivado synthesis report - zero BRAM utilisation (OOC) #798

Tests

• Added a new test, test_dense_unrolled that verifies dense resource layers implement avoiding zero multiplications are correct
• Comparison with "standard" Dense Resource will be shortly available in the (updated) PR hls4ml Optimization API [Part 1] #768.

Checklist

• I have read the guidelines for contributing.
• I have commented my code, particularly in hard-to-understand areas.
• I have made corresponding changes to the documentation.
• My changes generate no new warnings.
• I have installed and run pre-commit on the files I edited or added.
• I have added tests that prove my fix is effective or that my feature works.

[bo3z]

I will add pre-commit additionally, last time I ran it, some tests were broken, so will add it a subsequent commit.

[bo3z]

This is ready for review, seems that pre-commit can re-arrange the order of includes in C++ header files and it could cause compilation error.

[jmitrevs]

We merged part 1. Should we merge part 2?

[vloncar]

I'm reviewing it. Slowly 😃 . But it's next in line, then HGQ.

[jmitrevs]

The pytest error is unrelated to the PR so from my side this can be merged. I'll let Vladimir give the last OK.