-
Notifications
You must be signed in to change notification settings - Fork 31
Fix taxation of pensions and implement missing parameters of Wachstumschancengesetz #1093
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
Conversation
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
Codecov report seems like fake news to me, covered through the regression tests in |
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.
I did not check in minute detail, but it certainly looks impressive! Thanks a lot!
All of the above was just me trying to fix the docs; realized too late I can just build them locally. Note to self: Need to switch to |
Documentation build overview
Show files changed (5 files in total): 📝 5 modified | ➕ 0 added | ➖ 0 deleted
|
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 a couple of more detailed notes!
src/gettsim/tests_germany/policy_cases/full_taxes_and_transfers/2019-01-01/hh_id_2.yaml
Show resolved
Hide resolved
src/gettsim/tests_germany/policy_cases/wohngeld/2006-01-01/hh_id_15.yaml
Outdated
Show resolved
Hide resolved
### What problem do you want to solve? Add the inputs introduced in [1093](ttsim-dev/gettsim#1093).
What problem do you want to solve?
This turned from a simple renaming PR to a much larger one. What I changed:
There's one limitation I'm aware of: For each pension type, we have only one input variable. Because the Ertragsanteil depends on the age at which the pension is claimed, we cannot handle cases where someone get's two pensions of the same type, but claimed them in different years. But I doubt this will become relevant any time soon.
I have no good test cases for the taxation of pensions, so I just added some regression tests.