Skip to content

Commit 8455985

Browse files
committed
betters naming
1 parent 6024cf1 commit 8455985

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

climada/engine/option_appraisal/appraiser.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ class MeasuresAppraiser(RiskTrajectory):
5757
"averted risk",
5858
"risk",
5959
"measure net cost",
60-
"measure cost benefit",
60+
"benefit minus cost",
6161
]
6262

6363
def __init__(
@@ -123,7 +123,7 @@ def _generic_metrics(
123123
no_measures["reference risk"] = no_measures["risk"]
124124
no_measures["averted risk"] = 0.0
125125
no_measures["measure net cost"] = 0.0
126-
no_measures["measure cost benefit"] = 0.0
126+
no_measures["benefit minus cost"] = 0.0
127127
LOGGER.debug(f"Computing cash flow for: {metric_name}.")
128128
cash_flow_metrics = self._calc_per_measure_annual_cash_flows()
129129
LOGGER.debug(f"Merging with base metric: {metric_name}.")
@@ -136,7 +136,7 @@ def _generic_metrics(
136136

137137
averted_risk = base_metrics["averted risk"]
138138
cash_flow_metrics = base_metrics["measure net cost"]
139-
base_metrics["measure cost benefit"] = averted_risk - cash_flow_metrics
139+
base_metrics["benefit minus cost"] = averted_risk - cash_flow_metrics
140140

141141
if measures is not None:
142142
base_metrics = base_metrics.loc[
@@ -163,7 +163,7 @@ def subtract_no_measure(group, no_measure, merger):
163163
merger.append("coord_id")
164164

165165
return base_metrics.groupby(
166-
["group", "metric", "date"], group_keys=False, dropna=False
166+
["group", "metric", "date"], group_keys=False, dropna=False, observed=False
167167
).progress_apply(
168168
subtract_no_measure, no_measure=no_measures_metrics, merger=merger
169169
)
@@ -231,7 +231,7 @@ def _calc_per_measure_annual_cash_flows(self):
231231
def _calc_cb(self, base_metrics: pd.DataFrame):
232232
averted_risk = base_metrics["averted risk"]
233233
cash_flow_metrics = base_metrics["measure net cost"]
234-
base_metrics["measure cost benefit"] = averted_risk - cash_flow_metrics
234+
base_metrics["benefit minus cost"] = averted_risk - cash_flow_metrics
235235
return base_metrics
236236

237237
def _calc_waterfall_CB_plot_data(

0 commit comments

Comments
 (0)