fastmachinelearning / hls4ml

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

Additional cleanup of the codebase #750

Closed vloncar closed 1 year ago

vloncar commented 1 year ago

Description

Minor cleanups around the codebase:

Type of change

Tests

No need for extra tests, no new features were introduced

Checklist

jmitrevs commented 1 year ago

Is this for 0.7.0 or 0.8.0?

vloncar commented 1 year ago

Is this for 0.7.0 or 0.8.0?

0.7.0. It doesn't make any changes to the functionality, it just reorganizes things a little bit. so it should be safe. If this messed up some imports, we would either get circular imports (all tests fail), or failure to import qkeras stuff (qkeras tests fail). But we can see from the logs that there are no import warnings and all tests pass.