search

keras-team / keras-applications

Reference implementations of popular deep learning models.
Other
2k stars 910 forks source link

Duplicated Resnet50 code and weight files #74

Closed NivekNey closed 5 years ago

NivekNey commented 5 years ago

These are for include_top=False, the weight files are not pointing to the same weight file. They also differ in terms of layer naming. Is there a reason to keep them both? Also noticed tf.keras.applications is pointing to resnet50.ResNet50.

taehoonlee commented 5 years ago

@NivekNey, the resnet50.ResNet50 is a legacy and resnet.ResNet50(=resnet_common.ResNet50) is factorized over the ResNet variations and new one. But the reason why we are keeping the weight file is not to break the current usages. For example, someone, who are thresholding of softmax values of ResNet50, may get the different results if we are replacing the resnet50.ResNet50 with resnet.ResNet50.

NivekNey commented 5 years ago

Ok got it, I guess this is more a model zoo than exposed user endpoint, so it makes sense to keep both for people to choose/use. Thanks for the official reply!