Jovan Mitrevski

Results 30 issues of Jovan Mitrevski

# Description As discussed in #624, convert_from_keras_model with project_name set to nondefault value (e.g. mynewprojname) fails to build, because in build.tcl the variable ${mynewprojname} is not defined. This fixes the...

This is an attempt to fix quantized tanh and add quantized sigmoid activation support for QKeras ingestion. In Quartus the tanh handling probably needs an optimizer to rename it to...

## Prerequisites Please make sure to check off these prerequisites before submitting a bug report. - [x] Test that the bug appears on the current version of the master branch....

bug

# Description This is the main development to ingest QONNX in hls4ml. It's a bit of a large PR, and will probably take a while to review and update as...

The `normalize` function call in both `nnet_batchnorm.h` and `nnet_batchnorm_stream.h` has an interface of: ```c++ template void normalize( data_T data[CONFIG_T::n_in], res_T res[CONFIG_T::n_in], typename CONFIG_T::scale_t scale[CONFIG_T::n_in], typename CONFIG_T::bias_t bias[CONFIG_T::n_in] ) ``` However,...

# Description In the encoded convolution, the case when the in_width is smaller than the nominal min_width needs to be handled as a special case. This attempts to fix this...

bug

In the original implementation, inplace variables are created when initializing an object, which can run into problems if the source node is changed due to optimizers. This PR moves the...

bug

# Description I am opening this pull request more for discussion. If people agree with this change, the same change would also need to be made for the VivadoAccelator backend....

bug

# Description This change simply appends `applied_flows` to `ModelGraph._applied_flows` in the beginning, before it is filled, so that `ModelGraph._applied_flows` always has an accurate description of what has been applied. This...

bug

# Description This is the beginning of the oneAPI backend, and is just included here for easier sharing and discussion. It is not ready for merging. Currently only `io_parallel` Dense...

please test