keras-team / keras

Deep Learning for humans
http://keras.io/
Apache License 2.0
62.06k stars 19.48k forks source link

Add `SE-ResNet` to keras.applications #15268

Closed innat closed 1 year ago

innat commented 3 years ago

System information.

TensorFlow version (you are using): 2.5 Are you willing to contribute it (Yes/No): Yes. As much as possible.

Describe the feature and the current behavior/state.

Model like SE-ResNet( [18, 34, 50, 101, 152] ) are absent (IDK why) in keras application along with other models. So, I think, It would be a good addition to the keras core library.

Will this change the current api? How? Yes. It will change as follows

tensorflow.keras.applications.SEResNet18
tensorflow.keras.applications.SEResNet34
tensorflow.keras.applications.SEResNet50
tensorflow.keras.applications.SEResNet101
tensorflow.keras.applications.SEResNet152

Who will benefit from this feature? ML engineers and researcher who uses tf.keras.

Others

Others implementation: https://github.com/qubvel/classification_models

rchao commented 3 years ago

Thanks for opening the issue. Can you provide a paragraph as a proposal for justifying adding this model? If you can provide some context of how widely this is used or some citation, that would be helpful.