From e228d4d686b1d4bb67274fa40f57d9bcbf393bf8 Mon Sep 17 00:00:00 2001
From: Kristina Mazur <kristina.mazur@tum.de>
Date: Fri, 29 Nov 2024 18:25:36 +0100
Subject: [PATCH] Fix double accounting for unit conversion

---
 .../src/standard_mission/fuel_handling/fuel_handling.cpp       | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/mission_analysis/src/standard_mission/fuel_handling/fuel_handling.cpp b/mission_analysis/src/standard_mission/fuel_handling/fuel_handling.cpp
index c74fdabf..2b1c1859 100644
--- a/mission_analysis/src/standard_mission/fuel_handling/fuel_handling.cpp
+++ b/mission_analysis/src/standard_mission/fuel_handling/fuel_handling.cpp
@@ -298,8 +298,7 @@ double standard_mission::calculate_holding_fuel(const double& aircraft_mass, con
         EndnodeReadOnly<int>("energy_carrier_ID").read(rtIO->acxml.at("requirements_and_specifications/design_specification/propulsion/energy_provider/descent")).value();
     atmosphere holding_atm; //Airbus defines to calculate holding pattern at ISA-Conditions
     double holding_mach(EndnodeReadOnly<double>("/program_settings/general/fuel_planning/holding/holding_mach_number").read(this->rtIO->moduleConfig).value());
-    double holding_altitude(convertUnit(FOOT, METER,
-        EndnodeReadOnly<double>("/program_settings/general/fuel_planning/holding/holding_altitude").read(this->rtIO->moduleConfig).value()));
+    double holding_altitude(EndnodeReadOnly<double>("/program_settings/general/fuel_planning/holding/holding_altitude").read(this->rtIO->moduleConfig).value());
     if (EndnodeReadOnly<bool>("program_settings/general/fuel_planning/holding/use_economical_speed").read(this->rtIO->moduleConfig).value()) {
         this->calculate_econ_cruise_speed(&holding_mach, holding_altitude, 0., aircraft_mass, this->aero.cruiseConfiguration, "cruise");
     }
-- 
GitLab