Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ResNeXt [50, 101] to keras.applications #917

Closed
innat opened this issue Aug 28, 2021 · 11 comments
Closed

Add ResNeXt [50, 101] to keras.applications #917

innat opened this issue Aug 28, 2021 · 11 comments
Assignees

Comments

@innat
Copy link
Contributor

innat commented Aug 28, 2021

System information.

TensorFlow version (you are using): 2.5
Are you willing to contribute it (Yes/No) : Yes

Describe the feature and the current behavior/state.

ResNeXt is a well-known classification model that is weirdly missing from the keras.applications. The idea that is presented here is great and I think it's fit for the model member in the keras package.

Will this change the current api? How?
Yes.

from tensorflow.keras.applications.resnext50 import ResNeXt50
from tensorflow.keras.applications.resnext101 import ResNeXt101

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

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

@rchao
Copy link

rchao commented Sep 2, 2021

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.

@gcunhase
Copy link

gcunhase commented Jun 3, 2022

Any update on this?

@innat
Copy link
Contributor Author

innat commented Jun 3, 2022

This model was introduced in 2016 and now it's 2022 and still, we don't have it in keras.application even though it's cited by 6697. Same as its too.

@gcunhase
Copy link

gcunhase commented Jun 3, 2022

How can that be?

@gcunhase
Copy link

gcunhase commented Jun 3, 2022

@innat have you tried using ResNext from here? I'm not able to get good accuracies: qubvel/classification_models#73 .

@innat
Copy link
Contributor Author

innat commented Jun 3, 2022

I did use SE-ResNeXt most often. It worked well, comparable with DenseNet121.

@innat
Copy link
Contributor Author

innat commented Jun 7, 2022

cc @taehoonlee
Mentioning Lee who has once made a PR on ResNeXt HERE, keras-team/keras#11203.

@gcunhase
Copy link

gcunhase commented Jun 8, 2022

Thank you @innat !

@taehoonlee
Copy link

There are so many different types of deep learning architecture these days, (as I know) so François Chollet wants to add only those with special structural changes among them.

@innat
Copy link
Contributor Author

innat commented Jun 15, 2022

@taehoonlee Is the decision still persist? The torchvision provides these models. Models like wide-resnet, shuffle-net, unfortunately are missing as well.

@taehoonlee
Copy link

You can ask the maintainers. Now I am out of the keras team.

@tanzhenyu tanzhenyu transferred this issue from keras-team/keras Oct 13, 2022
@DavidLandup0 DavidLandup0 mentioned this issue Dec 17, 2022
2 tasks
@innat innat closed this as completed Aug 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants