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
Add tilt transformation #486
base: main
Are you sure you want to change the base?
Conversation
* added `tilt` function to `_transformations.py` * added it to the `index.rst` * created an example in `tilt.py` * added it to `__init__.py` * added the class `TestTilt` in `test_transformations.py`; however, the test is failling.
For some reason that I don't know the test of tilt passes when you change the sinal of g_z in `TestTilt:test_against_synthetic` in `test_transformations.py`
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this @leomiquelutti! Made a few comments but nothing major. Good work!
Co-authored-by: Leonardo Uieda <leo@uieda.com>
Do you know why the test for macOS for python 3.8 is failing and/or how to solve it, @leouieda? |
It was due to a failure when trying to push the coverage report to Codecov. I reran it and now it's passing. Nothing wrong with your code ❤️ |
Changes proposed:
tilt
function to_transformations.py
index.rst
tilt.py
__init__.py
TestTilt
intest_transformations.py
Some points/questions:
TestTilt::test_against_synthetic
is failing and I don't know why.tilt.py
require a title. I added but had a hard time with pyGMT, so I left them commented..py
file, comparing the transformations from TMI to the ones from RTP TMI. But that should be done in another PR.transformations.rst
?Can you please help me @santisoler.