andersbll / deeppy

Deep learning in Python
MIT License
1.38k stars 307 forks source link

Incorrect bprop() for LRN layer #15

Open donglaiw opened 9 years ago

donglaiw commented 9 years ago

In deeppy/feedforward/convnet_layers.py, the bprop for the LRN layer should not be the direct copy from the previous layer.

Reference: decaf https://github.com/UCB-ICSI-Vision-Group/decaf-release/blob/master/decaf/layers/cpp/local_response_normalization.cpp

andersbll commented 9 years ago

Agreed - this is a silly mistake! Thank you for reporting it.