fastmachinelearning / hls4ml

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

Update Jenkins test environment to avoid dependency hell #786

Closed vloncar closed 1 year ago

vloncar commented 1 year ago

Description

Jenkins tests are based on python 3.7 environment, in which we can install TF 2.11 as the latest version. When hls4ml is installed on top, onnx dependency causes the update of protobuf package (to 4.21) causing the environment to break and hls4ml (and tensorflow and qkeras) are unusable. We should consider updating the Python requirement to 3.8 for the next version of hls4ml to avoid this, but as an immediate band-aid solution to ensure tests run, we can install onnx==1.12 prior to hls4ml, that won't cause the update of protobuf and the environment works. we can update just the Jenkins environment to Python 3.8.

Type of change

Tests

Jenkins tests should run now, they fail on every branch currently.

Checklist

vloncar commented 1 year ago

After a discussion, updated the environment to python 3.8