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

[TESTS] Decrease test times by introducing testing model #6235

Merged
merged 1 commit into from
Aug 14, 2020

Conversation

tkonolige
Copy link
Contributor

In most unit tests, resnet is used as the default model for testing. As these are unit tests, they don't require a full sized model, and using resnet makes the run slowly. This PR introduces a new small synthetic model for use in testing. I've replaced most occurrences of resnet in the unit tests with this model. Time spent running tests/scripts/task_python_unittest.sh is 118.07s with this model vs 571.41s with resnet (on my Mac laptop, no gpu).

@tkonolige tkonolige force-pushed the faster_tests branch 10 times, most recently from 7d145c6 to abb2ea0 Compare August 11, 2020 23:08
@jroesch
Copy link
Member

jroesch commented Aug 12, 2020

It looks like you have some networking issue with the tracker, maybe try another CI run?

@tkonolige tkonolige force-pushed the faster_tests branch 2 times, most recently from 58361b8 to d685cfe Compare August 13, 2020 16:08
Adds a new testing model `tvm.relay.testing.synthetic` which is a small,
but representative model. Replaces resnet with this model in many tests.
@tkonolige
Copy link
Contributor Author

I seem to have fixed the sporadically failing tests too!

@jroesch
Copy link
Member

jroesch commented Aug 14, 2020

cc @areusch

@jroesch jroesch merged commit 37912a1 into apache:master Aug 14, 2020
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Aug 26, 2020
Adds a new testing model `tvm.relay.testing.synthetic` which is a small,
but representative model. Replaces resnet with this model in many tests.
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Aug 26, 2020
Adds a new testing model `tvm.relay.testing.synthetic` which is a small,
but representative model. Replaces resnet with this model in many tests.
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Aug 26, 2020
Adds a new testing model `tvm.relay.testing.synthetic` which is a small,
but representative model. Replaces resnet with this model in many tests.
trevor-m pushed a commit to trevor-m/tvm that referenced this pull request Sep 2, 2020
Adds a new testing model `tvm.relay.testing.synthetic` which is a small,
but representative model. Replaces resnet with this model in many tests.
trevor-m pushed a commit to neo-ai/tvm that referenced this pull request Sep 3, 2020
Adds a new testing model `tvm.relay.testing.synthetic` which is a small,
but representative model. Replaces resnet with this model in many tests.
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.

2 participants