-
Notifications
You must be signed in to change notification settings - Fork 31
Add unterhaltsvors implementation 2009 to 2016 #790
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 ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #790 +/- ##
==========================================
+ Coverage 89.85% 89.99% +0.13%
==========================================
Files 53 53
Lines 3834 3857 +23
==========================================
+ Hits 3445 3471 +26
+ Misses 389 386 -3 ☔ View full report in Codecov by Sentry. |
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.
Thanks again for implementing this!
Just one clarification question that a quick google search couldn't answer.
Regarding the tests: I will translate the official documents you linked to test files in the next days. For the other functions where only ifo comparisons are available, I will create regression tests.
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.
Looks good, thanks!
I added the tests based on the tables above.
@hmgaudecker do you want to have a last look? Also, the yamlfix hook fails here and on #756. A quick google search didn't yield a result, any idea?
Probably a bug in 1.16, seems like an autoupdate helped? |
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.
Great work, thanks!
Just some tiny stuff, which is only related to big pic questions, not newly introduced parameters here. Feel free to implement, @MImmesberger, or open a more general issue for that.
| out = mindestunterhalt[altersgrenzen[2]] - _kindergeld_erstes_kind_m | ||
| elif altersgrenzen[2] <= alter < altersgrenzen[3]: | ||
| out = mindestunterhalt[altersgrenzen[3]] - _kindergeld_erstes_kind_m | ||
| if mindestunterhalt[1]["min_alter"] <= alter <= mindestunterhalt[1]["max_alter"]: |
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 switched to the age thresholds of the Mindestunterhalt here, because the value of the transfer is taken from there as well. Makes it less error prone in the future imo.
Then, the age thresholds in the UV params file are not used after 2016, so I removed the 2017 reform from there.
Do you agree?
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.
Apologies this is dragging on -- I should have had a careful look earlier!
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.
Great work, thanks a lot @mjbloemer and @MImmesberger !!!
This PR implements Unterhaltsvorschuss for the years 2009 to 2016.
Closes #566
Closes #479
Implementation before 2009 will be a separate PR once this is done.
Tests/comparison
Test with synthetic household middle age group child (6-11)
_unterhaltsvors_anspruch_kind_m_2009_bis_2014_unterhaltsvors_anspruch_kind_m_2009_bis_2014_unterhaltsvors_anspruch_kind_m_2009_bis_2014_unterhaltsvors_anspruch_kind_m_2009_bis_2014_unterhaltsvors_anspruch_kind_m_2009_bis_2014_unterhaltsvors_anspruch_kind_m_2009_bis_2014_unterhaltsvors_anspruch_kind_m_2009_bis_2014_unterhaltsvors_anspruch_kind_m_2009_bis_2014_unterhaltsvors_anspruch_kind_m_anwendungsvors_unterhaltsvors_anspruch_kind_m_anwendungsvors_unterhaltsvors_anspruch_kind_m_2016_unterhaltsvors_anspruch_kind_m_2016_unterhaltsvors_anspruch_kind_m_ab_2017_unterhaltsvors_anspruch_kind_m_ab_2017_unterhaltsvors_anspruch_kind_m_ab_2017_unterhaltsvors_anspruch_kind_m_ab_2017_unterhaltsvors_anspruch_kind_m_ab_2017_unterhaltsvors_anspruch_kind_m_ab_2017_unterhaltsvors_anspruch_kind_m_ab_2017_unterhaltsvors_anspruch_kind_m_ab_2017_unterhaltsvors_anspruch_kind_m_ab_2017_unterhaltsvors_anspruch_kind_m_ab_2017Test with synthetic household, lower age group child (0-5)
*no rounding rule applied
Test with synthetic household, upper age group child (12-17), 0 gross income:
Test with synthetic household, upper age group child (12-17), 600 Euro gross income: