-
Notifications
You must be signed in to change notification settings - Fork 33
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
Ground track using cartopy #173
Comments
Hi Greg, we had a long weekend holiday here, hence the late reply. This looks like a nice piece of functionality again :) One question on the 'settings' input. Wouldn't it be easier to have each of the settings entries be an explicit input (with default) for the function itself? That way:
But, as you know, I am not that good a Python programmer, nor am I very familiar with common practices. Let me know what you think about the above suggestion |
Absolutely I will correct it soon! Super |
Hi again I have corrected the function and I believe now it could be more useful. I have also created a minimal working example so that you can test if everything is working as you wish
In addition to Tudat, the packages I hope the function will be useful in the future, and if not you can just ignore this current issue. Thank you for your time and attention Greg |
Hi Greg, somehow we lost track of this code your wrote, our bad! I think we can split this in two:
|
Dear Dominic
I will do my best to find this piece of code and restore it. It has probably been an year since the last time I opened it. This week I will try to conclude it. Unfortunately my week is super busy until Friday. In the weekend I can try to fix it and see if the code is still working.
Thank you for your time and attention
Best
Greg
… On 14 Dec 2022, at 14:20, DominicDirkx ***@***.***> wrote:
Hi Greg, somehow we lost track of this code your wrote, our bad! I think we can split this in two:
Add your new Python functions into Tudatpy
Add the functionality into an example application. Perhaps this one: tudat-team/tudatpy-examples#14 <tudat-team/tudatpy-examples#14> would be a good choice
—
Reply to this email directly, view it on GitHub <#173>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AQ76ADECCSZWSF5CH55IDYTWNHCTTANCNFSM5TP2ZN7A>.
You are receiving this because you authored the thread.
|
Hi Greg, no need to hurry! I didn't mean to ask you to do this, I think we can merge your new Python functions into Tudat already (if you want 'credit' in the tudatpy commit history, you can open a pull request, though). The use in the Galileo example will come when we properly write that example. |
Hey Dominic
Ok perfect. There is no need for the pull request. I am very happy that my code gets used and I hope it will serve its purpose. In case you need help, I will do my best to help you in time.
Super thank you and have a great day
Best
Greg
… On 15 Dec 2022, at 15:37, DominicDirkx ***@***.***> wrote:
Hi Greg, no need to hurry! I didn't mean to ask you to do this, I think we can merge your new Python functions into Tudat already (if you want 'credit' in the tudatpy commit history, you can open a pull request, though). The use in the Galileo example will come when we properly write that example.
—
Reply to this email directly, view it on GitHub <#173>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AQ76ADCK7SCXEUWGGXDU5YLWNMULBANCNFSM5TP2ZN7A>.
You are receiving this because you authored the thread.
|
Hi
I have developed a small piece of code that can visualize ground tracks using cartopy
The available list of map is listed in the documentation of the function.
I tried it and it works for me. You have to install cartopy from conda, but nothing else is required (apart from numpy, but this is pretty normal to have).
If you think it is useful, let me know.
I hope it will help
Have a super day!
The text was updated successfully, but these errors were encountered: