gluon-cv icon indicating copy to clipboard operation
gluon-cv copied to clipboard

A reimplementation of MixNet in Gluon.

Open becauseofAI opened this issue 5 years ago • 8 comments

Add a new model named MixNet to gluoncv model-zoo. It is a reimplementation of MixNet in Gluon. The paper is here. The model will be trained with ImageNet dataset when the machine is ready.

becauseofAI avatar Aug 07 '19 17:08 becauseofAI

Job PR-900-1 is done. Docs are uploaded to http://gluon-vision-staging.s3-website-us-west-2.amazonaws.com/PR-900/1/index.html Code coverage of this PR: pr.svg vs. Master: master.svg

mli avatar Aug 07 '19 17:08 mli

PR looks awesome. Minor things about removing commented lines. Looking forward to the numbers trained on ImageNet.

bryanyzhu avatar Aug 07 '19 19:08 bryanyzhu

Job PR-900-2 is done. Docs are uploaded to http://gluon-vision-staging.s3-website-us-west-2.amazonaws.com/PR-900/2/index.html Code coverage of this PR: pr.svg vs. Master: master.svg

mli avatar Aug 08 '19 03:08 mli

I'd like to hold this implementation from merged into our incoming release, unless one of the following can be met:

  1. the weights are transferred from the official source, or
  2. the network can be trained to match the accuracy.

The reason behind is due to MXNet has a limited implementation of split, which cannot split the channel dimension unevenly. So this might not be an identical re-implementation. Still, I'm open with further suggestions.

hetong007 avatar Aug 13 '19 15:08 hetong007

@hetong007 split may not support uneven division, but @becauseofAI used workaround which uses F.slice_axis with pre-calculated brakets. So I guess it's matching the original implementation? @becauseofAI Can you verify if you can convert weights and achieve same accuracy as reported in paper? 

zhreshold avatar Aug 14 '19 18:08 zhreshold

OK you are right, F.slice_axis makes it equivalent.

hetong007 avatar Aug 14 '19 18:08 hetong007

@becauseofAI any update on this?

zhreshold avatar Sep 20 '19 00:09 zhreshold

@becauseofAI any update on this?

djaym7 avatar Dec 26 '19 19:12 djaym7