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

simple but effective speed up of test_tau.py #1720

Merged
merged 1 commit into from Mar 14, 2017

Conversation

Projects
None yet
3 participants
@barsch
Member

barsch commented Mar 14, 2017

reducing runtime on my computer from 116s to 14s by randomly skipping test values - set speedup factor to 1 to have the original behaviour

simple but effective speed up of test_tau.py
reducing runtime on my computer from 116s to 14s by randomly skipping
test values - set speedup factor to 1 to have the original behaviour
@barsch

This comment has been minimized.

Show comment
Hide comment
@barsch

barsch Mar 14, 2017

Member

don't know if wanted - but I'm kind of annoyed by the runtime of the taup tests

Member

barsch commented Mar 14, 2017

don't know if wanted - but I'm kind of annoyed by the runtime of the taup tests

@krischer

This comment has been minimized.

Show comment
Hide comment
@krischer

krischer Mar 14, 2017

Member

Very good idea.

Member

krischer commented Mar 14, 2017

Very good idea.

@barsch

This comment has been minimized.

Show comment
Hide comment
@barsch

barsch Mar 14, 2017

Member

merging - failing test is not related

Member

barsch commented Mar 14, 2017

merging - failing test is not related

@barsch barsch closed this Mar 14, 2017

@barsch barsch reopened this Mar 14, 2017

@barsch barsch merged commit 26d0fb1 into master Mar 14, 2017

6 of 9 checks passed

codecov/changes 6 files have unexpected coverage changes not visible in diff.
Details
docker-deb-buildbot Deb packaging succeeded but tests failed
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
ci/circleci Your tests passed on CircleCI!
Details
codecov/patch 100% of diff hit (target 90%)
Details
codecov/project 87.58% (+1.54%) compared to 693e89a
Details
continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
docker-testbot Docker tests succeeded
Details

@barsch barsch deleted the taup_speedup branch Mar 14, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment