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

Fix flaky dcgan test by using fake data #1044

Merged
merged 1 commit into from Aug 24, 2022
Merged

Fix flaky dcgan test by using fake data #1044

merged 1 commit into from Aug 24, 2022

Conversation

hudeven
Copy link
Contributor

@hudeven hudeven commented Aug 23, 2022

The dcgan test is flaky as reported in #1038 and #1040
The root cause is that it's very slow to download the "classroom" dataset from http://dl.yf.io/

To make the test script more stable, let's use fake data in run_python_examples.sh. The test time for dcgan is reduced from minutes to ~5 seconds.

Test plan:

./run_python_examples.sh dcgan

image

@netlify
Copy link

netlify bot commented Aug 23, 2022

Deploy Preview for pytorch-examples-preview canceled.

Name Link
🔨 Latest commit f2030f6
🔍 Latest deploy log https://app.netlify.com/sites/pytorch-examples-preview/deploys/630568872a89870008b51a54

@hudeven hudeven added the testing related to unit / integration tests label Aug 23, 2022
Copy link

@ninginthecloud ninginthecloud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! thanks for identifying this flaky test and making changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla signed testing related to unit / integration tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants