search

broadinstitute / keras-rcnn

Keras package for region-based convolutional neural networks (RCNNs)
Other
555 stars 222 forks source link

Fix tensorflow backend tests #160

Closed milani closed 6 years ago

milani commented 6 years ago

The two commits try to fix #159 and also part of #158.

As described in the commit descriptions, I am not sure if the typecast was there on purpose and it made problems. Removing it does not seem to break the tests at least on my system.

milani commented 6 years ago

keras floatx is configuration dependent, I think. But this one should always use float32.

jhung0 commented 6 years ago

Why should it always use float32?

milani commented 6 years ago

@jhung0 Sorry I replied in the email and it appeared not in the correct place. I answered your question in the right thread.