Merge pull request #8 from jmitrevs/initialRecurr-keras

fix simple-rnn config for Keras; make test names unique

Author: JanFSchulte

hls4ml/backends/oneapi/passes/recurrent_templates.py

@@ -373,6 +373,9 @@ def format(self, node):
         )
         simple_rnn_params['recurrent_activation'] = 'relu'
 
+        # In Keras there is no recurrent bias, so put a placeholder
+        simple_rnn_params.setdefault('recurrent_bias_t', simple_rnn_params['bias_t'])
+
         simple_rnn_config = self.template.format(**simple_rnn_params)
 
         act_params = self._default_config_params(node)

test/pytest/test_qkeras.py

@@ -620,7 +620,7 @@ def test_qlstm(backend):
     config = hls4ml.utils.config_from_keras_model(
         model, granularity='name', default_precision="ap_fixed<8,1>", backend=backend
     )
-    output_dir = str(test_root_path / f'hls4mlprj_qkeras_qsimplernn_{backend}')
+    output_dir = str(test_root_path / f'hls4mlprj_qkeras_qlstm_{backend}')
     hls_model = hls4ml.converters.convert_from_keras_model(model, hls_config=config, output_dir=output_dir, backend=backend)
     hls_model.compile()