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

Thrust from maximum thrust and thrust coefficient #106

Closed
DominicDirkx opened this issue Dec 7, 2016 · 3 comments
Closed

Thrust from maximum thrust and thrust coefficient #106

DominicDirkx opened this issue Dec 7, 2016 · 3 comments

Comments

@DominicDirkx
Copy link
Member

In many applications, the use a thrust coefficients is typical for a (simplified) representation of the thrust magnitude, based on the current maximum thrust value.

@DominicDirkx
Copy link
Member Author

DominicDirkx commented Dec 13, 2016

A first step of these modifications have been added in the following branch

https://github.com/DominicDirkx/tudat/tree/SpacePlaneModifications

A unit test is included for a single set of dependencies of Isp and thrust. However, the unit tests should be extended with:

  • Unit test for multiple user-defined (guidance) input variables to the guidance
  • Unit test for 1 and 3 independent variables
  • Unit test for each of the available dependencies of the thrust (only Mach number and dynamic pressure tested; altitude, density and pressure are not)

Also, the coefficient file reader (see #104) should be interfaced with the ParameterizedThrustMagnitudeSettings class to allow tables with (maximum) thrust/specific impulse to be read from files.

Finally, as the functionality encapsulated in this feature is quite expansive and flexible, an example application would be very useful.

@DominicDirkx
Copy link
Member Author

The branch has been updated with some modifications. We decided to not allow multiple user-defined guidance parameters for now (a warning will be shown that this feature is not tested if it is used).

An indirect test of the 3-dimensional file-reading is now include through the test of control surface deflections (1-dimensional file reading remains untested). Remaining issues are:

  • Unit test for 1 independent variable
  • Unit test for each of the available dependencies of the thrust (only Mach number and dynamic pressure tested; altitude, density and pressure are not)

@DominicDirkx
Copy link
Member Author

The pull request (#113) for this code has now been added

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant