Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor Power Plant Costing Fixes #1418

Merged
merged 7 commits into from
Jul 19, 2024
Merged

Minor Power Plant Costing Fixes #1418

Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
6c6d419
minor fixes to ppcosting
bpaul4 May 28, 2024
f65dae9
Merge branch 'main' of https://github.com/IDAES/idaes-pse into minor-…
bpaul4 May 28, 2024
0a83891
Merge branch 'main' into minor-ppcosting-fixes
ksbeattie Jun 20, 2024
99a8435
correct additional COE docstring
bpaul4 Jun 20, 2024
cd9d797
Merge branch 'main' into minor-ppcosting-fixes
bpaul4 Jul 15, 2024
d5dd669
Merge branch 'main' into minor-ppcosting-fixes
bpaul4 Jul 18, 2024
1f828e4
Merge branch 'main' into minor-ppcosting-fixes
ksbeattie Jul 18, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
31 changes: 21 additions & 10 deletions idaes/models_extra/power_generation/costing/power_plant_capcost.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -326,11 +326,15 @@ def build_process_costs(
self.maintenance_material_cost / 12 # 1 month materials
+ self.non_fuel_and_waste_OC # 1 month nonfuel consumables
+ (
self.waste_costs_OC if waste is not None else 0 * CE_index_units
self.waste_costs_OC
if waste is not None
else 0 * CE_index_units / pyunits.year
) # 1 month waste
# inventory capital costs
+ (
self.fuel_cost_OC if fuel is not None else 0 * CE_index_units
self.fuel_cost_OC
if fuel is not None
else 0 * CE_index_units / pyunits.year
) # 60 day fuel supply
# Other costs
+ (
Expand Down Expand Up @@ -371,9 +375,10 @@ def build_process_costs(
)

self.additional_cost_of_electricity = Var(
initialize=70.9e-6,
initialize=0,
doc="additional cost to be added to the COE calculations"
+ " in millions",
+ " in millions; use 70.9e-6 for NGCC baseline report, "
+ " use 43.3e-6 for NGCC without carbon capture.",
units=CE_index_units / pyunits.MWh,
)

Expand All @@ -399,7 +404,7 @@ def build_process_costs(
if tonne_CO2_capture is not None:
if not hasattr(self, "tonne_CO2_capture"):
self.tonne_CO2_capture = Param(
initialize=tonne_CO2_capture, mutable=True, units=pyunits.ton
initialize=tonne_CO2_capture, mutable=True, units=pyunits.tonne
)
self.cost_of_capture = Expression(
expr=(
Expand All @@ -413,18 +418,24 @@ def build_process_costs(
)

if transport_cost is not None:
if isinstance(transport_cost, (Expression, ScalarExpression)) or (
isinstance(transport_cost, (Param, Var))
and transport_cost.get_units is None
if (
isinstance(transport_cost, (Expression, ScalarExpression))
or (isinstance(transport_cost, (Param, Var, float, int)))
) and (
pyunits.get_units(transport_cost) is None
or pyunits.get_units(transport_cost) == pyunits.dimensionless
):
self.transport_cost = (
transport_cost
* CE_index_units
/ pyunits.ton
/ pyunits.tonne
* self.tonne_CO2_capture
)
else:
self.transport_cost = transport_cost * self.tonne_CO2_capture
self.transport_cost = pyunits.convert(
transport_cost * self.tonne_CO2_capture,
to_units=CE_index_units,
)

else: # except the case where transport_cost is passed but tonne_CO2_capture is not passed
if transport_cost is not None:
Expand Down
Loading