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.

Sign up for GitHub

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

Jump to bottom

Propagators #25

Merged
merged 98 commits into from
Jun 1, 2016
Merged

Propagators #25

merged 98 commits into from
Jun 1, 2016

Conversation

DominicDirkx
Copy link
Member

New Propagation beta code

@DominicDirkx
Beta version of time-dependent sh gravity, without unit test
3873602
@DominicDirkx
Adding missing files
9ed8fcc
@DominicDirkx
First version of simulation setup, going to merge aerodynamics update…
9d4352f 
… branch
@DominicDirkx
Merge branch 'AerodynamicCoefficientInterface' into SimulationSetup
ceec142
@DominicDirkx
Merged with updated atmosphere models
aab8b47
@DominicDirkx
With preliminary flight conditions code from old setup
f359f8a
@DominicDirkx
Merge branch 'SimulationSetup' into FlightConditions
8631441
@DominicDirkx
Merge branch 'ShapeModels' into InterfaceSetup
855d6e9
@DominicDirkx
Merge branch 'RadiationPressureUpdate' into InterfaceSetup
f5b104d
@DominicDirkx
Intermediate commit for simulation setup
2f98922
@DominicDirkx
With extra setup functions
2565cea
@DominicDirkx
Merge branch 'FlightConditions' into InterfaceSetup
64a6863
@DominicDirkx
With extra acceleration model setup functions
d3df473
@DominicDirkx
With minor modifications
f03381e
@DominicDirkx
Some mroe progress on automatic setup of non-conservative forces
90d68a3
@DominicDirkx
Comments update
8d536bc
@DominicDirkx
Update createAccelerationModels.h
65e4054 
Fixed include guard.
@DominicDirkx
Working on unit tests
f341da7
@DominicDirkx
First part of unit test running
b8ca1ee
@DominicDirkx
Further modifications to interface setup
c7dc01c
@DominicDirkx
With extra comments and unit tests
f1f8224
@DominicDirkx
Updated unit tests
bfb5fd1
@DominicDirkx
Merge remote-tracking branch 'origin' into SimulationSetup
ccd2464
@DominicDirkx
Merge branch 'SimulationSetup' of https://github.com/DominicDirkx/tudat…
3129027 
… into SimulationSetup
@DominicDirkx
Merge remote-tracking branch 'SimulationSetup' into InterfaceSetup
3f4df4b
@DominicDirkx
Merge branch 'AerodynamicCoefficientInterface' into InterfaceSetup
c643c52
@DominicDirkx
Merge branch 'RadiationPressureUpdate' into InterfaceSetup
09f5d1a
@DominicDirkx
Merge branch 'ShapeModels' into InterfaceSetup
49e19d0
@DominicDirkx
Merged with existing branches
82d2afd
@DominicDirkx
Merge remote-tracking branch 'origin' into TimDepSh
7c632a1
@magnific0
Copy link
Member

@DominicDirkx can you explain the reason for removing Tudat/Astrodynamics/StateDerivativeModels. I can see how there is overlap between the functionality of your code and these old files, however, there is some code (example applications) that depends on this. Moreover, I can imagine that user applications will be broken as well.

@DominicDirkx
Copy link
Member Author

@magnific0 I deleted the Tudat/Astrodynamics/StateDerivativeModels foilder, as all functionality that was included in it has now been subsumed into the Propagators folder. I thought it would be less confusing to delete it. I have updated versions of the applications that I will create pull requests for once this is committed.

However, it might be a good idea to not delete the folder until all documentation etc. has been updated to the new structure.

This was referenced May 19, 2016
Closed
Merged
@DominicDirkx DominicDirkx mentioned this pull request May 25, 2016
Merged
DominicDirkx and others added 4 commits May 27, 2016 14:08
@DominicDirkx
With default spherical harmonic fields
6a88a46
@DominicDirkx
Unit test fix
e8d93c3
@DominicDirkx
Streamlining of propagator settings (removed requirement for redundan…
0146e53 
…t map)
@magnific0
Bring back state derivative models (for now).
5967a59 
Fix include statement.

Fix GravityFieldVariations unit test, add kernel.

Codecheck finish checking

Add CSPICE clauses

Create subset of huge jup310 kernel using spkmerge
@magnific0
Copy link
Member

Hi @DominicDirkx

My changes are all very minor. We already discussed brinigng back the old statederivative models (for now).

Based on this I had a few general remarks, but I'll open seperate issues for that.

The addition looks very good. I'm happy to see such big improvements in the usability of Tudat.

@DominicDirkx DominicDirkx merged commit f2e3f55 into Tudat:master Jun 1, 2016
@DominicDirkx DominicDirkx deleted the Propagators branch November 17, 2016 21:01
@gviavattene gviavattene mentioned this pull request Sep 21, 2017
Closed
@ifodde ifodde mentioned this pull request Aug 3, 2018
Merged
DominicDirkx added a commit to DominicDirkx/tudat that referenced this pull request Jan 25, 2022
@DominicDirkx
Merge pull request Tudat#25 from tudat-team/feature/vector_6d_interpo…
2504f60 
…lator_python_fix

Feature/vector 6d interpolator python fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@magnific0 magnific0

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@DominicDirkx @magnific0