Skip to content

Use standard day SL density for VEAS in FGInitialCondition.#909

Merged
agodemar merged 1 commit intoJSBSim-Team:masterfrom
bcoconni:Fix_VEAS_in_FGInitialCondition
May 10, 2023
Merged

Use standard day SL density for VEAS in FGInitialCondition.#909
agodemar merged 1 commit intoJSBSim-Team:masterfrom
bcoconni:Fix_VEAS_in_FGInitialCondition

Conversation

@bcoconni
Copy link
Member

@bcoconni bcoconni commented May 8, 2023

Following the bug fix that has been released by @seanmcleod in PR #900, this PR applies the same fix to the FGInitialCondition class (it is not using the standard day SL density either for the computation of the equivalent airspeed).

I took this opportunity to check that FGAtmosphere::GetDensitySL() was not called elsewhere and remove the member FGAtmosphere::in.DensitySL that is no longer needed. In the process I also removed the member FGAtmosphere::in.PressureSL which is not used anywhere.

@seanmcleod
Copy link
Member

Good catch.

@codecov
Copy link

codecov bot commented May 8, 2023

Codecov Report

Merging #909 (b4282d2) into master (db830c6) will decrease coverage by 0.02%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master     #909      +/-   ##
==========================================
- Coverage   23.16%   23.15%   -0.02%     
==========================================
  Files         167      167              
  Lines       19644    19639       -5     
==========================================
- Hits         4551     4547       -4     
+ Misses      15093    15092       -1     
Impacted Files Coverage Δ
src/FGFDMExec.cpp 41.32% <ø> (-0.09%) ⬇️
src/models/FGAuxiliary.h 84.90% <ø> (ø)
src/initialization/FGInitialCondition.cpp 45.04% <50.00%> (-0.14%) ⬇️

@bcoconni
Copy link
Member Author

bcoconni commented May 9, 2023

Good catch.

@agodemar If this PR looks good to you as well then I guess you can push the button 😃

@agodemar agodemar merged commit 4445a75 into JSBSim-Team:master May 10, 2023
@bcoconni bcoconni deleted the Fix_VEAS_in_FGInitialCondition branch May 12, 2023 12:06
bcoconni added a commit to bcoconni/jsbsim that referenced this pull request May 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants