Method get_weight_information in backend_utils produced wrong WEIGHTS_START_INDEX array

pg020196 / Neural-Network-Translator

This repository contains a neural network translator which can translate neural network models to native code.

MIT License

8 stars 4 forks source link

Method get_weight_information in backend_utils produced wrong WEIGHTS_START_INDEX array #87

Closed ChristophBrandl closed 4 years ago

ChristophBrandl commented 4 years ago

weights_start_index-string: const uint16_t WEIGHTS_START_INDEX[4] = {0,0,0,26912.0};

weights array: const float WEIGHTS[26368] = {-0.033059194684028625,-0.18036584556102753,...};

issues:

26912.0 is a float
26912.0 is greater than 26368 (number of weights)

There has to be a issue in the calculation of the weights_start_index array.

ChristophBrandl commented 4 years ago

This issue is a consequence of issue #86 . In the above example the output height of the previous layer is 210.25. correct would be an output height of 196. This leads to an index out of the bounds of the weights array. With the correct output height the calculation of the start index array would be correct. Therefore this issue will be closed and is going to be fixed when issue #86 is resolved.