Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Contributing VGG models to model zoo #11830

Merged
merged 1 commit into from
Jul 21, 2018
Merged

Contributing VGG models to model zoo #11830

merged 1 commit into from
Jul 21, 2018

Conversation

abhinavs95
Copy link
Contributor

Description

Contributing VGG models to gluon model zoo - 16, 16_bn, 19, 19_bn. Models are trained with Gluon using script from here.

Checklist

Essentials

  • Changes are complete (i.e. I finished coding on this PR)
  • To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

  • Updated docs to reflect new accuracies and sources
  • Updated hashes for the model files

@abhinavs95 abhinavs95 requested a review from szha as a code owner July 19, 2018 22:44
@abhinavs95
Copy link
Contributor Author

@hetong007 could you please review and approve? Thanks!

@szha szha requested a review from hetong007 July 19, 2018 22:58
@hetong007
Copy link
Contributor

Please also push your scripts and logs to gluoncv, we'd like to have results reproducible.

@abhinavs95
Copy link
Contributor Author

@hetong007 I used the script train_imagenet.py from gluoncv for training.

Also, I only have the train command containing the hyper parameters and the accuracy plots with me, not the full training logs. The results can be reproduced using the same script and hyperparameters.

@hetong007
Copy link
Contributor

Please have your training command pushed to gluon-cv at: https://github.com/dmlc/gluon-cv/blob/master/docs/model_zoo/index.rst

Or you can also post it here and I'll verify it. Thanks.

@zhreshold zhreshold merged commit e14b575 into apache:master Jul 21, 2018
KellenSunderland pushed a commit to KellenSunderland/incubator-mxnet that referenced this pull request Jul 21, 2018
@hetong007 hetong007 mentioned this pull request Jul 23, 2018
2 tasks
XinYao1994 pushed a commit to XinYao1994/incubator-mxnet that referenced this pull request Aug 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants