Skip to content
This repository has been archived by the owner on Mar 14, 2021. It is now read-only.

WIP: Basic propagator #48

Closed
wants to merge 31 commits into from
Closed

WIP: Basic propagator #48

wants to merge 31 commits into from

Conversation

AlexS12
Copy link
Owner

@AlexS12 AlexS12 commented Dec 27, 2019

Objective

Implement a propagator using on OrdinaryDiffEq

  • Able to work for every implemented dynamic system
    • Able to work with SixDOFEulerFixedMass
    • Able to work with SixDOFQuaternionFixedMass
    • Once these systems have been settled I will implement the others
  • Capable of choosing different solvers
    • test
  • Capable of tunning solver parameters
    • test
  • Able to integrate considering aircraft forces and moments constant for every time step. User is responsible of the time step and will not benefit from stepsize control *yet.
  • Controls will be set using set_controls_trimmer, so no generalization of FCS will be performed yet.
  • Implement some typical inputs, such as step, doublet, ramp and harmonic (Implement some typical FCS inputs #57)
  • Test integration against Steven's results for F16
  • Test constant trimmed inputs produce coherent outputs for simple cases
  • Review of trimmer (trimmer fixes #55)
  • Documentation for propagator.jl and results_store.jl

@AlexS12 AlexS12 added the enhancement New feature or request label Dec 27, 2019
@codecov
Copy link

codecov bot commented Dec 27, 2019

Codecov Report

Merging #48 into master will increase coverage by 6.43%.
The diff coverage is 76.36%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #48      +/-   ##
==========================================
+ Coverage   79.79%   86.22%   +6.43%     
==========================================
  Files          31       33       +2     
  Lines        1148     1198      +50     
==========================================
+ Hits          916     1033     +117     
+ Misses        232      165      -67
Impacted Files Coverage Δ
src/models/Models.jl 100% <ø> (ø) ⬆️
src/aircrafts/trimmer.jl 96.36% <100%> (ø) ⬆️
src/models/dynamic_systems.jl 78.94% <40%> (+78.94%) ⬆️
src/models/results_store.jl 47.05% <47.05%> (ø)
src/models/propagator.jl 96.87% <96.87%> (ø)
src/models/propulsion.jl 80% <0%> (+2.5%) ⬆️
src/models/environment.jl 72.22% <0%> (+2.77%) ⬆️
src/models/aircraft.jl 73.91% <0%> (+4.34%) ⬆️
... and 5 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1b585d3...df89da5. Read the comment docs.

@AlexS12
Copy link
Owner Author

AlexS12 commented Apr 4, 2020

pushed force because some changes that were in this branch have been cherry picked in different PRs as they were not directly related to this feature:

#50 #51 #52 #53 #54

@AlexS12 AlexS12 linked an issue Apr 13, 2020 that may be closed by this pull request
@AlexS12 AlexS12 mentioned this pull request May 5, 2020
14 tasks
@AlexS12
Copy link
Owner Author

AlexS12 commented May 5, 2020

closing in favour of #65

@AlexS12 AlexS12 closed this May 5, 2020
@AlexS12 AlexS12 deleted the basic_propagator branch May 5, 2020 09:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setup equations of motion with DifferentialEquations.jl?
1 participant