`activityColWrite` is never set for SubArray class

neurosim / DNN_NeuroSim_V1.3

Benchmark framework of compute-in-memory based accelerators for deep neural network (inference engine focused)

62 stars 36 forks source link

`activityColWrite` is never set for SubArray class #12

Closed sepehrMSP closed 2 years ago

sepehrMSP commented 2 years ago

Hi, activityColWrite is variable that is used in many modules in the code. However, it is not set for SubArray class. Therefore its value is set to 0 by default and SubArray and its modules that use this variable will have wrong latency and power. @neurosim @alu75

neurosim commented 2 years ago

Hi! In this version (inference engine focused), activityColWrite is not truly in use because writing/reprogramming is not necessary. The weights are assumed to be written into the memory arrays in advance, and only reading is required for the computation. While in DNN_NeuroSim_V2.1 (on-chip training focused), the activityColWrite is in use for weight reprogramming.