Merge pull request #120 from DoubleML/p-fix-pension-example

Minor change to Pension notebook

Author: PhilippBach

.github/workflows/deploy_docu_dev.yml

@@ -5,7 +5,7 @@ name: Deploy Docu (dev)
 on:
   push:
     branches:
-      - main
+      - dev
   workflow_dispatch:
 
 

.github/workflows/test_build_docu_dev.yml

@@ -6,9 +6,11 @@ on:
   push:
     branches:
       - main
+      - dev
   pull_request:
     branches:
       - main
+      - dev
   workflow_dispatch:
     inputs:
       doubleml-py-branch:

.github/workflows/test_build_docu_released.yml

@@ -6,9 +6,11 @@ on:
   push:
     branches:
       - main
+      - dev
   pull_request:
     branches:
       - main
+      - dev
   workflow_dispatch:
   schedule:
     - cron: "0 9 * * 1,3,5"

doc/examples/R_double_ml_pension.ipynb

@@ -1141,7 +1141,7 @@
    },
    "source": [
     "These estimates that flexibly account for confounding are\n",
-    "substantially attenuated relative to the baseline estimate (*19559*) that does not account for confounding. They suggest much smaller causal effects of 401(k) eligiblity on financial asset holdings. The best model with lowest RMSE in both equations is the PLR model estimated via lasso. It gives the following estimate:"
+    "substantially attenuated relative to the baseline estimate (*19559*) that does not account for confounding. They suggest much smaller causal effects of 401(k) eligiblity on financial asset holdings."
    ]
   },
   {

doc/examples/py_double_ml_pension.ipynb

@@ -774,12 +774,13 @@
    ]
   },
   {
+   "attachments": {},
    "cell_type": "markdown",
    "id": "541d51d3",
    "metadata": {},
    "source": [
     "These estimates that flexibly account for confounding are\n",
-    "substantially attenuated relative to the baseline estimate (*19559*) that does not account for confounding. They suggest much smaller causal effects of 401(k) eligiblity on financial asset holdings. The best model with lowest RMSE in both equations is the PLR model estimated via lasso. It gives the following estimate:"
+    "substantially attenuated relative to the baseline estimate (*19559*) that does not account for confounding. They suggest much smaller causal effects of 401(k) eligiblity on financial asset holdings."
    ]
   },
   {