SSD example has missing files

[lvaleriu]

Havent you forgotten to add the files related to the scale layer?

Here you can see them in the original mxnet-ssd repository.

Thanks!

[piiswrong]

it is modified and these files are no longer needed.
See if you can run the version in mxnet/exmaple

[lvaleriu]

I already tried and it doesnt work without the scale layers. I added them manually and rebuild mxnet and it worked afterwards.

[piiswrong]

@zhreshold

[zhreshold]

These layers are not required any more here. What error did you get? @lvaleriu

[zhreshold]

Well, I want to make it clear that in order to comply the mxnet example style, I made several changes here in the example/ssd, thus the model is now slightly incompatible with the original repo. You will need to modify .params and .json in order to transfer model between these two repos now. However I don't recommend to do this.

The divergence could be solved when I finally make a change to the original repo. I'm still reluctant to do so because breaking the compatibility in the same repo is not a good idea.

[lvaleriu]

You are right. I've messed up the code by including the old weights with
some old train file execution. Sorry for that. So you've gived up your
Scale layer with an existing one from mxnet?

On Sun, Nov 13, 2016 at 7:54 PM, Joshua Z. Zhang [email protected]
wrote:

Well, I want to make it clear that in order to comply the mxnet example
style, I made several changes here in the example/ssd, thus the model is
now slightly incompatible with the original repo. You will need to modify
.params and .json in order to transfer model between these two repos now.
However I don't recommend to do this.

The divergence could be solved when I finally make a change to the
original repo. I'm still reluctant to do so because breaking the
compatibility in the same repo is not a good idea.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#3805 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA4o_yzxU9OXWlTy6bUX57EmTx5jTiSaks5q910AgaJpZM4KwdtI
.

[zhreshold]

Yes, the Scale layer was replaced by broadcast_mul.

[yajiedesign]

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!