Skip to content

Commit 01f049b

Browse files
authored
Merge pull request #1331 from PolicyEngine/fix/parameter-bug
Fix/parameter bug
2 parents db7aaaa + 636df44 commit 01f049b

3 files changed

Lines changed: 12 additions & 8 deletions

File tree

policyengine_uk/simulation.py

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -70,12 +70,6 @@ def __init__(
7070
if reform is not None:
7171
scenario = Scenario.from_reform(reform)
7272

73-
# Apply parametric reforms here
74-
75-
if scenario is not None:
76-
if scenario.parameter_changes is not None:
77-
self.apply_parameter_changes(scenario.parameter_changes)
78-
7973
self.branch_name = "default"
8074
self.invalidated_caches = set()
8175
self.debug: bool = False
@@ -136,6 +130,8 @@ def __init__(
136130
self.baseline = self.clone()
137131

138132
if scenario is not None:
133+
if scenario.parameter_changes is not None:
134+
self.apply_parameter_changes(scenario.parameter_changes)
139135
if scenario.simulation_modifier is not None:
140136
scenario.simulation_modifier(self)
141137

policyengine_uk/variables/contrib/policyengine/employer_ni/baseline_employer_cost.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,11 @@ def formula(person, period, parameters):
3535
)
3636

3737
# Calculate baseline employer cost
38-
baseline_parameters = parameters(period).baseline
38+
if person.simulation.baseline is None:
39+
return 0
40+
baseline_parameters = (
41+
person.simulation.baseline.tax_benefit_system.parameters(period)
42+
)
3943
baseline_class_1 = (
4044
baseline_parameters.gov.hmrc.national_insurance.class_1
4145
)

policyengine_uk/variables/contrib/policyengine/employer_ni/employer_ni_fixed_employer_cost_change.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,11 @@ def formula(person, period, parameters):
4242
)
4343

4444
# Calculate baseline employer cost
45-
baseline_parameters = parameters(period).baseline
45+
if person.simulation.baseline is None:
46+
return 0
47+
baseline_parameters = (
48+
person.simulation.baseline.tax_benefit_system.parameters(period)
49+
)
4650
baseline_class_1 = (
4751
baseline_parameters.gov.hmrc.national_insurance.class_1
4852
)

0 commit comments

Comments
 (0)