Skip to content
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

[NNVM] Add symbol squeezenet #1436

Merged
merged 5 commits into from
Jul 15, 2018
Merged

[NNVM] Add symbol squeezenet #1436

merged 5 commits into from
Jul 15, 2018

Conversation

merrymercy
Copy link
Member

Add squeezenet symbol in test. This is mainly used for performance benchmark.

@merrymercy
Copy link
Member Author

This PR also fixes the CI error of #1435 #1432

@tqchen tqchen merged commit 18ac95c into apache:master Jul 15, 2018
@TaoLv
Copy link
Member

TaoLv commented Jul 18, 2018

Any instructions about how to run the performance benchmark of this squeezenet as well as other imagenet models? I would like to evalute the performance of this models on Intel CPUs. Thanks.

@tqchen
Copy link
Member

tqchen commented Jul 18, 2018

@TaoLv please open a thread at https://discuss.tvm.ai/

@TaoLv
Copy link
Member

TaoLv commented Jul 19, 2018

Thanks, @tqchen. Since the description of this PR says it's mainly for performance benchmarking, so I think it's proper to ask for the usage of it here. Anyway, I have created a topic in the forum. https://discuss.tvm.ai/t/how-to-benchmark-the-imagenet-models-in-tvm-nnvm/484

@tqchen
Copy link
Member

tqchen commented Jul 19, 2018

No problem @TaoLv :) The reason we prefer opening threads vs discussing via issue/in PR is that PR get can get ignored and issues can get closed. Posting things on forum will allow others to participate and learn from the topic

@merrymercy merrymercy deleted the squeezenet branch August 3, 2018 23:55
tqchen pushed a commit to tqchen/tvm that referenced this pull request Aug 4, 2018
sergei-mironov pushed a commit to sergei-mironov/tvm that referenced this pull request Aug 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants