Assorted fixes (#845)

* Fix Remove unnecessary `baseline` parameter node  #844 and Original FRS simulations error due to no corporate wealth #843

* Remove docker build action for now

.github/workflows/pr.yml

@@ -57,17 +57,3 @@ jobs:
         run: make documentation
         env:
           POLICYENGINE_GITHUB_MICRODATA_AUTH_TOKEN: ${{ secrets.POLICYENGINE_GITHUB_MICRODATA_AUTH_TOKEN}}
-  Docker:
-    name: Docker
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout repo
-        uses: actions/checkout@v3
-      - name: Log in to the Container registry
-        uses: docker/login-action@v1
-        with:
-          registry: ghcr.io
-          username: ${{ github.actor }}
-          password: ${{ secrets.GITHUB_TOKEN }}
-      - name: Build container
-        run: docker build docs/streamlit -t ghcr.io/policyengine/policyengine-uk-documentation

.github/workflows/push.yml

@@ -79,25 +79,6 @@ jobs:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           BRANCH: gh-pages  # The branch the action should deploy to.
           FOLDER: docs/book/_build/html  # The folder the action should deploy.
-  Docker:
-    if: |
-      (github.repository == 'PolicyEngine/policyengine-uk')
-      && (github.event.head_commit.message == 'Update PolicyEngine UK')
-    name: Docker
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout repo
-        uses: actions/checkout@v3
-      - name: Log in to the Container registry
-        uses: docker/login-action@v1
-        with:
-          registry: ghcr.io
-          username: ${{ github.actor }}
-          password: ${{ secrets.GITHUB_TOKEN }}
-      - name: Build container
-        run: docker build docs/streamlit -t ghcr.io/policyengine/policyengine-uk-documentation
-      - name: Push container
-        run: docker push ghcr.io/policyengine/policyengine-uk-documentation
   Publish:
     if: |
       (github.repository == 'PolicyEngine/policyengine-uk')

changelog_entry.yaml

@@ -0,0 +1,5 @@
+- bump: patch
+  changes:
+    fixed:
+    - Bug causing frs_2021 simulations to error.
+    - Unnecessary system initialisation code.

policyengine_uk/system.py

@@ -37,7 +37,6 @@ class CountryTaxBenefitSystem(TaxBenefitSystem):
     def __init__(self, reform=None):
         super().__init__(entities, reform=reform)
 
-        self.parameters.add_child("baseline", self.parameters.clone())
         self.parameters = backdate_parameters(self.parameters, "2021-01-01")
 
 

policyengine_uk/variables/household/wealth/corporate.py

@@ -11,7 +11,10 @@ class shareholding(Variable):
 
     def formula(household, period, parameters):
         wealth = household("corporate_wealth", period)
-        if hasattr(household.simulation, "dataset"):
+        if (
+            hasattr(household.simulation, "dataset")
+            and household("corporate_wealth", period).sum() != 0
+        ):
             weight = household("household_weight", period)
             return wealth / (wealth * weight).sum()
         wealth = parameters(period).household.wealth

setup.py

@@ -33,7 +33,7 @@
         "microdf_python>=0.3.0",
         "pandas",
         "plotly>=4.14.3",
-        "PolicyEngine-Core>=2.11.3,<3",
+        "PolicyEngine-Core>=2.19.0,<3",
         "pyyaml>=5.3.1",
         "pytest",
         "requests>=2.25.1",