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

Update effnet.py #1448

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update effnet.py #1448

wants to merge 1 commit into from

Conversation

karthikayan4u
Copy link

I guess as keras has been updated with tensorflow, efficientnet.keras is no longer available and hence I am updating this line.

@ha-nso-li
Copy link
Contributor

I think effnet.py used efficientnet pip package, neither core of keras or tensorflow.
Is the implementation of tf.keras compatible?

@karthikayan4u
Copy link
Author

I think effnet.py used efficientnet pip package, neither core of keras or tensorflow.
Is the implementation of tf.keras compatible?

Actually there is a problem. We can't use pretrained weights as tf comes with weights itself and I don't know how to integrate the imagenet weights. Rest of all is fine.

@hgaiser
Copy link
Contributor

hgaiser commented Sep 3, 2020

What issue are you having? I'm guessing it is related to qubvel/efficientnet#127 ? This should be fixed with qubvel/efficientnet#128 . When that gets released, efficientnet should work with the latest tensorflow again.

@karthikayan4u
Copy link
Author

Yeah the same was the issue along with another..I couldn't use the pretrained iamgenet weights too!

@gosha20777
Copy link

I use the qubvel/efficientnet package and personally know the author of this package. This package works nicely and is well compatible with the latest TF versions.
I believe there is no need to replace this package. Plus it breaks compatibility.

@gosha20777
Copy link

In addition, I think it would be nice to replace the original resnet with the resnet from this package. This can solve many problems and will allow you to abandon the keras code and completely move to TF.

@hsahin hsahin changed the base branch from master to main June 17, 2021 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants