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

Removing overconstraints in off-design aircraft #134

Merged
merged 5 commits into from
Feb 21, 2025

Conversation

ngomezve
Copy link
Contributor

When TASOPT aircraft were analyzed off-design in fly_off_design!, the user was free to independently set the Mach number, altitude, and lift coefficient at cruise. In general, this leads to $L \neq W$. This PR changes fly_off_design! so that the cruise altitude is calculated for a desired lift coefficient.

@ngomezve
Copy link
Contributor Author

Addresses Issue #133

Copy link

codecov bot commented Feb 20, 2025

Codecov Report

Attention: Patch coverage is 96.55172% with 1 line in your changes missing coverage. Please review.

Project coverage is 79.14%. Comparing base (55cd54b) to head (2cdfba9).
Report is 6 commits behind head on main.

Files with missing lines Patch % Lines
src/mission/off_design.jl 92.30% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #134   +/-   ##
=======================================
  Coverage   79.13%   79.14%           
=======================================
  Files          82       82           
  Lines       13575    13585   +10     
=======================================
+ Hits        10743    10752    +9     
- Misses       2832     2833    +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ngomezve ngomezve requested a review from askprash February 20, 2025 16:31
simplified atmos docs
@argonaut22 argonaut22 added the bug Something isn't working label Feb 20, 2025
Copy link
Member

@askprash askprash left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@ngomezve ngomezve merged commit e182855 into MIT-LAE:main Feb 21, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants