Reduce test duration - Githubissues

keras-team / keras-applications

Reference implementations of popular deep learning models.

Other

2k stars 907 forks source link

Reduce test duration #25

Closed RaphaelMeudec closed 6 years ago

RaphaelMeudec commented 6 years ago

In this PR:

Fix of a small error in test_mobilenet: selected module was always mobilenet even if selected app was mobilenet_v2.MobileNetV2
Regroup inception_v3 and inception_resnet_v2 in a random.choice(INCEPTION_LIST) as each network tests take 300s and 120s to run

RaphaelMeudec commented 6 years ago

Here are two screenshots from the Travis tasks: test_inception is running either in ~120s or ~300s, depending on whether inception_resnet_v2 or inception_v3 is tested.

inceptionresnetv2

inceptionv3

taehoonlee commented 6 years ago

@RaphaelMeudec, Thank you for the PR.

The test_mobilenet revision LGTM.
Even if the separate tests for InceptionResNetV2 and InceptionV3 are time-consuming, I think it is right to leave them as they are now. The models are using different sub-modules while DenseNet (or NASNet) variations are using the same sub-modules with different parameters.

RaphaelMeudec commented 6 years ago

@taehoonlee I removed the commit associated with InceptionResnetV2 and InceptionV3.