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

Automatically sync .ipynb files between tudat-space and tudatpy-examples #109

Open
sbcowan opened this issue Feb 17, 2023 · 1 comment
Open
Assignees

Comments

@sbcowan
Copy link
Collaborator

sbcowan commented Feb 17, 2023

Problem: It is tedious to update all the example applications with new code updates, and manually sync them between two repos.

Proposed solution: Using Repo File Sync Action from GitHub, you can sync files across repos. This should not require too much time to implement and would accelerate the process of updating example applications for Tudat.

This becomes even more efficient if you don't also have to convert .ipynb files to .py files.

@larshinueber
Copy link

Hi @sbcowan,
I've had a look at how to sync the notebooks between tudatpy-examples and tudat-space. The Repo File Sync Action looks like a good option, but there would still be two notebook files in different repositories.
Instead, I thought we could import tudatpy-examples directly as a submodule in tudat-space. This way, we avoid having to keep two files in sync and it is unambiguous where files should be maintained. The disadvantage with this approach is that there is some overhead with maintaining the submodule (the submodule commit needs to be kept up-to-date, although this might be automated with another Github action, see this SO post).

What do you think? I have created a draft PR with the proposed changes (#148), if you have any feedback or concerns with the approach, please let me know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: tudat-space
Development

No branches or pull requests

2 participants