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

adjust tolerance for farfield test case #1681

Merged
merged 3 commits into from Feb 23, 2017

Conversation

Projects
None yet
3 participants
@megies
Member

megies commented Feb 22, 2017

After merging #1642 there's a test fail in one older debian docker testbox:

Looks like a simple precision issue to me. @seisman maybe you can double-check (once our docker testbot has run through)

@seisman

It's OK for me.

Show outdated Hide outdated obspy/core/tests/test_event.py Outdated
@krischer

This comment has been minimized.

Show comment
Hide comment
@krischer

krischer Feb 23, 2017

Member

Looks good to me :) Let's wait for the docker tests before merging.

Member

krischer commented Feb 23, 2017

Looks good to me :) Let's wait for the docker tests before merging.

@megies

This comment has been minimized.

Show comment
Hide comment
@megies

megies Feb 23, 2017

Member

CI is all green.. gonna do a squash-and-merge for this on..

Member

megies commented Feb 23, 2017

CI is all green.. gonna do a squash-and-merge for this on..

@megies megies merged commit 38527c6 into maintenance_1.0.x Feb 23, 2017

5 checks passed

continuous-integration/appveyor/branch AppVeyor build succeeded
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
docker-deb-buildbot Deb packaging and testing succeeded
Details
docker-testbot Docker tests succeeded
Details

@megies megies deleted the fix_farfield_test branch Feb 23, 2017

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