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

Print result in center_of_mass test #242

Merged
merged 2 commits into from Dec 3, 2015

Conversation

Projects
None yet
1 participant
@leouieda
Member

leouieda commented Dec 3, 2015

The doctest in fatiando.gravmag.tensor.center_of_mass was checking the
result (a numpy array) without printing it with a given precision. This
was causing the test to fail unpredictably in different computers.

The test now prints the result with .1f. This should fix the failures.

Fixes #241

Checklist:

  • Make tests for new code
  • Create/update docstrings
  • Include relevant equations and citations in docstrings
  • Code follows PEP8 style conventions
  • Code and docs have been spellchecked
  • Include new dependencies in docs, requirements.txt, README, and .travis.yml
  • Documentation builds properly
  • All tests pass
  • Can be merged
  • Changelog entry (leave for last)
Print result in center_of_mass test
The doctest in fatiando.gravmag.tensor.center_of_mass was checking the
result (a numpy array) without printing it with a given precision. This
was causing the test to fail unpredictably in different computers.

The test now prints the result with `.1f`. This should fix the failures.

Fixes #241

leouieda added a commit that referenced this pull request Dec 3, 2015

Merge pull request #242 from fatiando/center_of_mass-test
Print result in center_of_mass test

@leouieda leouieda merged commit e2cacd1 into master Dec 3, 2015

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@leouieda leouieda deleted the center_of_mass-test branch Dec 3, 2015

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