apache / mxnet

Lightweight, Portable, Flexible Distributed/Mobile Deep Learning with Dynamic, Mutation-aware Dataflow Dep Scheduler; for Python, R, Julia, Scala, Go, Javascript and more
https://mxnet.apache.org
Apache License 2.0
20.79k stars 6.79k forks source link

Inconsistencies in Gluon Model Zoo API #13661

Open ChaiBapchya opened 5 years ago

ChaiBapchya commented 5 years ago

Currently, if I were to use Gluon Model Zoo and use any model for eg DenseNet, I face 2 issues regarding APIs a. Lack of clarity about differences in the APIs (Titlecase vs lowercase) E.g. mxnet.gluon.model_zoo.vision.AlexNet(classes=1000, **kwargs) and mxnet.gluon.model_zoo.vision.alexnet(pretrained=False, ctx=cpu(0), root='/home/jenkins_slave/.mxnet/models', **kwargs)

b. Even if that were clear, APIs themselves weren't consistent with function definitions for few networks

By inconsistency I mean, **kwargs instead of mentioning all the parameters that are required by the function. (e.g. pretrained=False, ctx=cpu(0))

Case in point - mobile nets, densenets, resnets, squeezenets (note the case sensitivity of these APIs)

Compared to the likes of alexnet and inceptions

mxnet-label-bot commented 5 years ago

Hey, this is the MXNet Label Bot. Thank you for submitting the issue! I will try and suggest some labels so that the appropriate MXNet community members can help resolve it.

ChaiBapchya commented 5 years ago

@mxnet-label-bot add [API change]