-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Endogenous Bedarfsgemeinschaften #778
base: main
Are you sure you want to change the base?
Conversation
I'm not too happy with the current implementation because I needed to copy+paste a lot of code in regarding SGB II Vermögensgrenzen and KiZ calculation (we need them on But it may be a good idea to postpone this to another PR (if its possible at all), this one is already quite complicated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just skimmed through a couple of things to keep the conversation up!
…h_id if there are multiple FGs in one HH.
…s if necessary).
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #778 +/- ##
==========================================
+ Coverage 89.69% 90.05% +0.35%
==========================================
Files 53 53
Lines 3796 3951 +155
==========================================
+ Hits 3405 3558 +153
- Misses 391 393 +2 ☔ View full report in Codecov by Sentry. |
Sorry that this comes super late: While working on the test files, I noticed a major bug that is not easy to fix and would lead to jumps in disposable income. The reason is that Kindergeldübertrag depends on the amount of Wohngeld, but the Kindergeldübertrag currently uses a "candidate Wohngeld" function as an input, not the actual amount. This is not easy to fix inside the DAG. I think it doesn't make sense to introduce even more candidate functions to make this work given that we discussed to move to several GETTSIM calls in the future. Hence, I would vote for not merging this one. We can use the test files and the logic developed in this PR in the PR that tackles this. |
Closes #763, #758, #622
Here is a DAG of this PR that I'll update if there are any changes:
![dagitty-model-5](https://private-user-images.githubusercontent.com/74215010/346321723-5fd97d2b-9194-45b3-9c22-ce322546b1b3.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzk2MDY4ODYsIm5iZiI6MTczOTYwNjU4NiwicGF0aCI6Ii83NDIxNTAxMC8zNDYzMjE3MjMtNWZkOTdkMmItOTE5NC00NWIzLTljMjItY2UzMjI1NDZiMWIzLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMTUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjE1VDA4MDMwNlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTUwMjU4MzZlYTZlYTQxN2UzNzhlOTI5NTZlZmI0MTE2ZmRkNTE4Y2VmYmRmOTA5ZjA0MzIxYjNjNTM5YzFiMTYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.NkhK0BONjs5hI4TDmeRh2lgbH9BNDjYny8mBVmItjPw)
Code to reproduce DAG on DAGitty