fastmachinelearning / hls4ml

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

Remove tf_to_hls #795

Closed vloncar closed 1 year ago

vloncar commented 1 year ago

Description

This PR removes tf_to_hls. The tf_to_hls converter is outdated, unused, not exposed through the main API and based on old approach that likely won't work anymore. If we decide to support lower-level tensorflow API alongside Keras, this approach would be different anyway, so there's no reason to keep this around.

A question to be addressed: this converter introduced BiasAdd as a temporary layer and the associated optimizer. If ONNX and PyTorch converters don't plan to use it, it should be removed in this PR as well.

Type of change

Tests

Nothing to test.

Checklist

jmitrevs commented 1 year ago

I think the pytest failure is transient. The test succeed on my computer.