Closed LL-Geo closed 2 years ago
💖 Thanks for opening your first pull request! 💖
Please make sure you read the following:
A few things to keep in mind:
make format
to make sure your code follows our style guide.@LL-Geo thanks for this! I'll try have a look at it as soon as possible.
I like the figure! Did you make it youself?
I like the figure! Did you make it youself?
Haha. Yepp. Find a new hobby for making figures :laughing:
@LL-Geo great stuff! Left a few suggestions to make sure the website builds correctly. The tutorial itself is really good 💯
You can commit the suggestions directly through GitHub (there should be a "Commit suggestion" button next to each). Once you're done, please give me a ping and I'm happy to merge this ❤️
Please also add yourself to the AUTHORS.md file if you'd like. Either in this PR or in a new PR later would be fine. Whatever works best for you.
All done! Haha. Thanks for all the comments and suggestions! :wink:
🎉🎉🎉 Congrats on merging your first pull request and welcome to the team! 🎉🎉🎉
If you would like to be added as a author on the Zenodo archive of the next release, make sure that you have added your full name, affiliation, and ORCID (optional) to the AUTHORS.md
file of this repository.
We hope that this was a good experience for you. Let us know if there is any way that the contributing process could be improved.
🎉 Congrats on the first PR, @LL-Geo! Hopefully the first of many 🙂
Add tutorials for Converting geodetic latitude and height into geocentric latitude and radius
I feel it's better to tell the coordinate and the ellipsoid height first, then go to the gravity disturbance
Reminders:
make format
andmake check
to make sure the code follows the style guide.doc/api/index.rst
and the base__init__.py
file for the package.AUTHORS.md
file (if you haven't already) in case you'd like to be listed as an author on the Zenodo archive of the next release.