Merge pull request #397 from NREL/chp-default-bug

Chp default bug
NREL · May 22, 2024 · 80c0d62 · 80c0d62
2 parents ab53b8a + 235972b
commit 80c0d62
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -23,6 +23,10 @@ Classify the change according to the following categories:
     ### Deprecated
     ### Removed
 
+## Develop
+### Fixed
+- Convert `max_electric_load_kw` to _Float64_ before passing to function `get_chp_defaults_prime_mover_size_class`
+
 ## v0.46.1
 ### Changed
 - Updated the GHP testset .json `./test/scenarios/ghp_inputs.json` to include a nominal HotThermalStorage and ColdThermalStorage system.

diff --git a/src/core/chp.jl b/src/core/chp.jl
@@ -207,7 +207,7 @@ function CHP(d::Dict;
     # For non-heating CHP, Prime Generator, defaults require electric load metrics
     if !isempty(electric_load_series_kw)
         avg_electric_load_kw = sum(electric_load_series_kw) / length(electric_load_series_kw)
-        max_electric_load_kw = maximum(electric_load_series_kw)
+        max_electric_load_kw = convert(Float64, maximum(electric_load_series_kw))
     else
         avg_electric_load_kw = nothing
         max_electric_load_kw = nothing

diff --git a/test/runtests.jl b/test/runtests.jl
@@ -22,7 +22,7 @@ elseif "CPLEX" in ARGS
     end
 
 else  # run HiGHS tests
-
+    
     @testset "Inputs" begin
         @testset "hybrid profile" begin
             electric_load = REopt.ElectricLoad(;