Turicreate: Image Similarity exported coreml spec includes two useless layer

[jakesabathia2]

Coreml model spec for image similarity has two useless layer:
softmax and fc1, which doesn't effect the result but should be removed.

[srikris]

@nickjong @jakesabathia2 Is this a 6.0 ship blocker?

[nickjong]

@jakesabathia2 Is this a regression from 5.8?

[jakesabathia2]

The perf I believe isn't affected.
But 5.8 didn't have those two useless layers.

[srikris]

@jakesabathia2 Actually that makes sense now. Now, we are simply just using the coreml-ml model as is and it has those 2 layers. We should remove them.

[srikris]

Here is s sample output of model exported via TC 5.8 and 6.0. The interfaces are the same but the 2 layers does add some space to the .mlmodel but CoreML optimizes unused layers so it won't have impact on the app.

[srikris]

@nickjong @syoutsey Clearing this out of 6.0.

[kritikadusad]

@TobyRoseman, I would like to work on this issue. Thank you!

[TobyRoseman]

@kritikadusad - ok, great. I've assigned this issue to you. Let me know if you have any questions.