New config_from_keras_model (#690)

* Don't set duplicate attributes

* Expand the use of backend-specific attributes

* Reorganize attributes a bit to remove duplicates

* Move string conversion utilities to new module

* Remove unnecessary init for activation attributes

* New config_from_keras_model function

* Remove config from parsers (no converters use it)

* Save activation's quantizer config in the parser

* Use keras parsing fn to build the initial config

* Lowercase 'trace' attribute (removes duplicate)

* Fix tests that fail due to config update

* Update Jenkinsfile

* Use ChoiceAttribute for convolution strategy

* Try loading a different libdl.so on newer Linux boxes

* Fix test output dirs

* To future self: I'm sorry for this, but it was necessary

* black + isort compatability

Co-authored-by: Javier Duarte <[email protected]>

Author: vloncar

.pre-commit-config.yaml

@@ -2,15 +2,15 @@ exclude: ^hls4ml\/templates\/(vivado|quartus)\/(ap_types|ac_types)\/
 
 repos:
 - repo: https://github.com/psf/black
-  rev: 22.10.0
+  rev: 22.12.0
   hooks:
   - id: black
     language_version: python3
     args: ['--line-length=125',
            '--skip-string-normalization']
 
 - repo: https://github.com/pre-commit/pre-commit-hooks
-  rev: v4.3.0
+  rev: v4.4.0
   hooks:
   - id: check-added-large-files
   - id: check-case-conflict
@@ -24,18 +24,19 @@ repos:
   - id: trailing-whitespace
 
 - repo: https://github.com/PyCQA/isort
-  rev: 5.10.1
+  rev: 5.11.4
   hooks:
   - id: isort
+    args: ["--profile", "black", --line-length=125]
 
 - repo: https://github.com/asottile/pyupgrade
-  rev: v3.1.0
+  rev: v3.3.1
   hooks:
   - id: pyupgrade
     args: ["--py36-plus"]
 
 - repo: https://github.com/asottile/setup-cfg-fmt
-  rev: v2.1.0
+  rev: v2.2.0
   hooks:
   - id: setup-cfg-fmt
 
@@ -46,10 +47,10 @@ repos:
     exclude: docs/conf.py
     additional_dependencies: [flake8-bugbear, flake8-print]
     args: ['--max-line-length=125',  # github viewer width
-           '--extend-ignore=E203']  # E203 is not PEP8 compliant
+           '--extend-ignore=E203,T201']  # E203 is not PEP8 compliant
 
 - repo: https://github.com/mgedmin/check-manifest
-  rev: "0.48"
+  rev: "0.49"
   hooks:
   - id: check-manifest
     stages: [manual]

Jenkinsfile

@@ -15,7 +15,7 @@ pipeline {
         dir(path: 'test') {
           sh '''#!/bin/bash --login
               conda activate hls4ml-py37
-              pip install tensorflow
+              pip install tensorflow pyparsing
               pip install -U ../ --user
               ./convert-keras-models.sh -x -f keras-models.txt
               pip uninstall hls4ml -y'''