ClassyVision icon indicating copy to clipboard operation
ClassyVision copied to clipboard

A minor documentation issue in efficientnet

Open vinayak015 opened this issue 3 years ago • 1 comments

📚 Documentation

efficientnet.py

It's a clean and well documented code. Kudos to the authors and thank you for making it open source :)

On the line 195, I think it's just an Excitation layer. Squeeze is being done by AdaptiveAvgPool2d.

vinayak015 avatar Dec 22 '21 15:12 vinayak015

Glad that you find the implementation useful @vinayak015! You're right, this only corresponds to the "Excitation" part of the SE op. I think the code generally referred to this section as corresponding to the SE layer, but please feel free to send a PR adding a qualifier that this is the Excitation part of the SE layer :)

mannatsingh avatar Dec 24 '21 16:12 mannatsingh