Closed jdebacker closed 2 months ago
@rickecon I'm running the full suite of tests locally tonight. If those pass, I believe this PR is ready to merge.
With the latest commits, I only have 4 test failures locally:
FAILED tests/test_txfunc.py::test_txfunc_est[DEP] - assert False
FAILED tests/test_txfunc.py::test_txfunc_est[GS] - assert False
FAILED tests/test_txfunc.py::test_tax_func_loop - assert False
FAILED tests/test_txfunc.py::test_tax_func_estimate - assert False
These are expected as we know the optimizer for the tax function estimation returns difference results with difference version of Python/SciPy.
Attention: Patch coverage is 68.19788%
with 90 lines
in your changes missing coverage. Please review.
Project coverage is 71.40%. Comparing base (
dc0860a
) to head (f3b21f2
).
@jdebacker. I reviewed this PR and everything looks good. I just submitted a PR to your branch that updates the version number of OG-Core in setup.py
and in ogcore/__init__.py
and adds numba
to setup.py
and requirements.txt
. As soon as you make a decision on that PR, we can merge this.
@rickecon I just merged your PR. Thanks for thinking of those housekeeping items.
This PR adds 3 additional pension systems (in additional the default US Social Security system):