-
Notifications
You must be signed in to change notification settings - Fork 5.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
enable alexnet benchmark #6852
enable alexnet benchmark #6852
Conversation
'num_class': num_class, | ||
'is_infer': is_infer, | ||
'num_samples': num_samples | ||
} | ||
define_py_data_sources2( | ||
"train.list", None, module="provider", obj="process", args=args) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
groups=2 if use_mkldnn else 1
net = img_conv_layer(
input=net, filter_size=5, num_filters=256, stride=1, padding=2, groups=groups)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个我刚好改了逻辑,就是你说的现在这样了。thx。
@@ -79,6 +79,7 @@ fi | |||
# inference benchmark | |||
for use_mkldnn in True False; do | |||
for batchsize in 1 2 4 8 16; do | |||
infer alexnet group2 $batchsize $use_mkldnn | |||
infer googlenet v1 $batchsize $use_mkldnn | |||
infer resnet 50 $batchsize $use_mkldnn | |||
infer vgg 19 $batchsize $use_mkldnn |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里的测试顺序和train的刚好反过来,麻烦都换成一样的测试顺序吧。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
没问题
infer vgg 19 $batchsize $use_mkldnn | ||
infer resnet 50 $batchsize $use_mkldnn | ||
infer googlenet v1 $batchsize $use_mkldnn | ||
infer alexnet 2 $batchsize $use_mkldnn |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
openblas的脚本里面,是没有$use_mkldnn的
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! My fault.
@@ -36,4 +36,5 @@ for batchsize in 64 128 256; do | |||
train vgg 19 $batchsize | |||
train resnet 50 $batchsize | |||
train googlenet v1 $batchsize | |||
train alexnet 2 $batchsize $use_mkldnn |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里多了一个$use_mkldnn
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks to point out. Done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
enable alexnet benchmark training and inference on MKL and openblas