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

Reduce time cost of test_operator.test_norm, test_operator. test_laop_5 and test_sparse_operator.test_elemwise_binary_ops #16067

Merged
merged 1 commit into from
Sep 3, 2019

Conversation

gyshi
Copy link
Contributor

@gyshi gyshi commented Sep 2, 2019

because test_norm and test_elemwise_binary_ops use rand_shape_2d()。 by default, rand_shape_2d generate shape(0,0) to (10, 10), so it will generate too large shape. for some operator, it will consume much time. so i limit it .

for test_laop_5, i modify for loop from (1, 10) to (1, 5), beacuse i it's ok! so the test time from 71.902s to 16.569s.

@gyshi
Copy link
Contributor Author

gyshi commented Sep 2, 2019

@haojin2 , thx for reviewing

@haojin2 haojin2 changed the title fix some test files test time is too long Reduce time cost of test_operator.test_norm and test_sparse_operator.test_elemwise_binary_ops Sep 2, 2019
@haojin2 haojin2 requested review from eric-haibin-lin and haojin2 and removed request for eric-haibin-lin September 2, 2019 05:20
@haojin2 haojin2 self-assigned this Sep 2, 2019
@gyshi gyshi changed the title Reduce time cost of test_operator.test_norm and test_sparse_operator.test_elemwise_binary_ops Reduce time cost of test_operator.test_norm, test_operator. test_laop_5 and test_sparse_operator.test_elemwise_binary_ops Sep 2, 2019
@haojin2 haojin2 merged commit f195098 into apache:master Sep 3, 2019
gyshi added a commit to gyshi/incubator-mxnet that referenced this pull request Sep 7, 2019
gyshi added a commit to gyshi/incubator-mxnet that referenced this pull request Sep 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants