Skip to content
Snippets Groups Projects

aerodynamic_analysis (schueltke): corrected wave drag calibration and neutral point estimation

Merged Florian Schültke requested to merge fix/aerodynamic_analysis_corrections into develop
All threads resolved!

Description

  • corrected wave drag calibration (the calibration is only applied, when the uncalibrated polar shows wave drag)
  • corrected function call of neutral point estimation with correct trimmed polar

Related Issue(s)

N.A

Other Changes

N.A

Screenshots/Logs

N.A

Testing Instructions

  1. download this branch and build aerodynamic_analysis
  2. run module on already converged aircraft design
  3. the output of drag polars for a delta stabilizer angle of 0 deg should be 0 in wave drag for the whole Ma=0.2 and Ma=0.5 polar. First wave drag occurrences from Ma=0.6 on (before also for Ma=0.2 and Ma=0.5 wave drag was calculated)
  4. the neutral point should be equivalent to the design mass CG now for a trimmed polar (static_margin of 0)

Developer Checklist

  • Code has been tested locally and/or in pipeline.
  • Merge conflicts resolved with the target branch.

Additional Notes

N.A

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Florian Schültke requested changes

    requested changes

  • Florian Schültke resolved all threads

    resolved all threads

  • Florian Schültke aborted the automatic merge because the source branch was updated. Learn more.

    aborted the automatic merge because the source branch was updated. Learn more.

  • added 1 commit

    • e485dfa4 - reverted change as discussed --> to be clarified for final release

    Compare with previous version

  • No wave drag at Mach numbers < 0.6 :thumbsup:
    I think we need to have a look at the wave drag calibration for comparibly "low" Mach numbers (around Mach 0.6- 0.7) but that is something for later release.

  • Maren Huxel approved this merge request

    approved this merge request

  • merged

  • Maren Huxel mentioned in commit d7bc4831

    mentioned in commit d7bc4831

  • Please register or sign in to reply
    Loading