Skip to content

Commit

Permalink
fix: make iteration loops for simplified train consistent (#265)
Browse files Browse the repository at this point in the history
  • Loading branch information
olelod authored Nov 1, 2023
1 parent 8105e3b commit baeee5c
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 6 deletions.
5 changes: 0 additions & 5 deletions src/libecalc/core/models/compressor/train/simplified_train.py
Original file line number Diff line number Diff line change
Expand Up @@ -560,11 +560,6 @@ def calculate_maximum_rate_given_outlet_pressure_single_calculation_point(
" This should normally not happen. Please contact eCalc support."
)

maximum_actual_volume_rate = maximum_rate_function(
heads=polytropic_head,
extrapolate_heads_below_minimum=False,
)

return maximum_actual_volume_rate


Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,7 @@ def test_calculate_maximum_rate_given_outlet_pressure_all_calculation_points(
pressure_ratios = [1, 2, 3, 4, 5, 10, 100, 1000]

# These expected max rates are here just to assure stability in the results. They are not assured to be correct!
approx_expected_max_rates = [1116990, 1359255, 1536130, 1052085, 1052085, 1052085, 1052085, 1052085]
approx_expected_max_rates = [1116990, 1359022, 1536130, 1052085, 1052085, 1052085, 1052085, 1052085]
for pressure_ratio, approx_expected_max_rate in zip(pressure_ratios, approx_expected_max_rates):
calculated_max_rate = (
CompressorTrainSimplifiedKnownStages.calculate_maximum_rate_given_outlet_pressure_all_calculation_points(
Expand Down

0 comments on commit baeee5c

Please sign in to comment.