Skip to content

Commit 59b28d0

Browse files
Fix stimulation cost tooltip text incorrect indirect cost - actual value is 5%, not 15%. Update tooltip text for clarity and to reference cost per injection well parameter.
1 parent cfbcb9f commit 59b28d0

File tree

2 files changed

+11
-5
lines changed

2 files changed

+11
-5
lines changed

src/geophires_x/Economics.py

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1619,7 +1619,7 @@ def __init__(self, model: Model):
16191619
)
16201620

16211621
# TODO https://github.com/NREL/GEOPHIRES-X/issues/383?title=Parameterize+indirect+cost+factor
1622-
contingency_and_indirect_costs_tooltip = 'plus 15% contingency plus 12% indirect costs'
1622+
stimulation_contingency_and_indirect_costs_tooltip = 'plus 15% contingency plus 5% indirect costs'
16231623

16241624
# noinspection SpellCheckingInspection
16251625
self.Cstim = self.OutputParameterDict[self.Cstim.Name] = OutputParameter(
@@ -1628,11 +1628,17 @@ def __init__(self, model: Model):
16281628
PreferredUnits=CurrencyUnit.MDOLLARS,
16291629
CurrentUnits=CurrencyUnit.MDOLLARS,
16301630
ToolTipText=f'Default correlation: ${self.stimulation_cost_per_injection_well.value}M '
1631-
f'per injection well {contingency_and_indirect_costs_tooltip}. '
1632-
f'Provide {self.ccstimadjfactor.Name} to multiply the default correlation. '
1633-
f'Provide {self.ccstimfixed.Name} to override the default correlation and set your own cost.'
1631+
f'per injection well {stimulation_contingency_and_indirect_costs_tooltip}. '
1632+
f'Provide {self.stimulation_cost_per_injection_well.Name} to set the correlation '
1633+
f'cost per injection well. '
1634+
f'Provide {self.ccstimadjfactor.Name} to multiply the correlation-calculated cost. '
1635+
f'Provide {self.ccstimfixed.Name} to override the correlation and set your own '
1636+
f'total stimulation cost.'
16341637
)
16351638

1639+
# TODO https://github.com/NREL/GEOPHIRES-X/issues/383?title=Parameterize+indirect+cost+factor
1640+
contingency_and_indirect_costs_tooltip = 'plus 15% contingency plus 12% indirect costs'
1641+
16361642
# See TODO re:parameterizing indirect costs at src/geophires_x/Economics.py:652
16371643
# (https://github.com/NREL/GEOPHIRES-X/issues/383)
16381644
self.Cexpl = self.OutputParameterDict[self.Cexpl.Name] = OutputParameter(

src/geophires_x_schema_generator/geophires-result.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -351,7 +351,7 @@
351351
"Drilling and completion costs per redrilled well": {},
352352
"Stimulation costs": {
353353
"type": "number",
354-
"description": "Default correlation: $1.25M per injection well plus 15% contingency plus 12% indirect costs. Provide Reservoir Stimulation Capital Cost Adjustment Factor to multiply the default correlation. Provide Reservoir Stimulation Capital Cost to override the default correlation and set your own cost.",
354+
"description": "Default correlation: $1.25M per injection well plus 15% contingency plus 5% indirect costs. Provide Reservoir Stimulation Capital Cost per Injection Well to set the correlation cost per injection well. Provide Reservoir Stimulation Capital Cost Adjustment Factor to multiply the correlation-calculated cost. Provide Reservoir Stimulation Capital Cost to override the correlation and set your own total stimulation cost.",
355355
"units": "MUSD"
356356
},
357357
"Stimulation costs (for redrilling)": {},

0 commit comments

Comments
 (0)