Skip to content
This repository has been archived by the owner on Dec 22, 2021. It is now read-only.

add warnings to inform that SRTomo is not real tomography #394

Merged
merged 1 commit into from
Apr 24, 2017

Conversation

rafaelmds
Copy link
Contributor

@rafaelmds rafaelmds commented Apr 24, 2017

Fixes #371

This PR includes warnings about the SRTomo class informing that is not a real tomography.

Checklist:

  • Make tests for new code (at least 80% coverage)
  • Create/update docstrings
  • Include relevant equations and citations in docstrings
  • Docstrings follow the style conventions
  • Code follows PEP8 style conventions
  • Code and docs have been spellchecked
  • Include new dependencies in doc/install.rst, environment.yml, ci/requirements.txt.
  • Documentation builds properly (run cd doc; make locally)
  • Changelog entry (leave for last to avoid conflicts)

@leouieda
Copy link
Member

@rafaelmds thanks for doing this! Merging.

@leouieda leouieda merged commit 85de276 into fatiando:master Apr 24, 2017
@leouieda leouieda added the docs label Apr 24, 2017
@leouieda leouieda added this to the 0.6 milestone Apr 24, 2017
@rafaelmds
Copy link
Contributor Author

Hello Leo,

doesn't it need a changelog entry for v0.6?
I was waiting automatic tests to push that commit.

@leouieda
Copy link
Member

Oops, sorry. That's OK, this is only a documentation change so no need for a changelog entry.

@rafaelmds rafaelmds deleted the SRTomo-warning branch April 28, 2017 14:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants