titu1994 / DenseNet

DenseNet implementation in Keras
MIT License
707 stars 294 forks source link

no longer works with newest keras #45

Closed jcpeterson closed 6 years ago

jcpeterson commented 6 years ago

Get ImportError: cannot import name _obtain_input_shape with keras 2.2.2 as opposed to 2.2.0

titu1994 commented 6 years ago

Hmm I gotta fix that. Is it called something else ?

jcpeterson commented 6 years ago

Not sure yet. Reverted for now

jcpeterson commented 6 years ago

I think you fixed this on contrib. It's keras_application instead of keras.application or something

jcpeterson commented 6 years ago

did meant to close this

titu1994 commented 6 years ago

Fixed via https://github.com/titu1994/DenseNet/commit/70185f37273a361cedf563f90aba8b5e5c039dcc

jcpeterson commented 6 years ago

Thanks. Just curious -- did you test this with 2.2.2?

titu1994 commented 6 years ago

Yes. I have 2.2.2 installed locally and I was able to run the inference script properly v

jcpeterson commented 6 years ago

Great