Skip to content
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

Low-thrust porkchop example, radiation pressure update, some other fixes #40

Merged
merged 7 commits into from Feb 23, 2024

Conversation

alopezrivera
Copy link
Contributor

@alopezrivera alopezrivera commented Jan 23, 2024

This PR contains the following updates:

Astro:

  • Shape-based low-thrust porkchop example including trajectory visualizations to verify the porkchops, as well as some discussion on further steps to design low-thrust trajectories.
  • Updated all radiation pressure models used in the examples to the new interface

Infra:
All changes implemented in the a new script -create_scripts.py- which substitutes clean_py_notebooks.py and convert_py_to_ipynb

  • Improved the generation of Python scripts from Jupyter notebooks
    • Script creation (using Jupyter nbconvert) and cleanup automatically done from create_scripts.py
    • Removed a few instances of syntax errors caused by the cleanup of the generated Python scripts (the process which used to live in clean_py_notebooks.py)
  • Syntax verification of examples: added step in create_scripts.py syntax check warning of generated Python scripts
  • Functional verification of examples: added option in in create_scripts.py to check all Python scripts (those with no syntax errors)

@alopezrivera alopezrivera changed the title Low-thrust porkchop example Low-thrust porkchop example, radiation pressure update, some other fixes Feb 20, 2024
@alopezrivera
Copy link
Contributor Author

DO NOT MERGE YET: awaiting verification of pygmo examples

@DominicDirkx DominicDirkx merged commit ac56a32 into tudat-team:master Feb 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants