PSLmodels / Tax-Brain

Tax-Brain is an integrator model for PSL tax models
http://taxbrain.pslmodels.org/
MIT License
8 stars 12 forks source link

update API docs #186

Closed jdebacker closed 2 years ago

jdebacker commented 2 years ago

This PR fixes issue where the API sections are listed twice in the Jupyter Book (see the left sidebar here). This was a result of some changes in the more recent JB releases.

andersonfrailey commented 2 years ago

Now that #181 is merged, these test failures should go away when you merge this with the master branch.

jdebacker commented 2 years ago

@andersonfrailey Can you try rerunning the checks here? It looks like there was just a timeout when installing some packages.

codecov-commenter commented 2 years ago

Codecov Report

Merging #186 (181bed6) into master (180063a) will decrease coverage by 0.19%. The diff coverage is 80.53%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #186      +/-   ##
==========================================
- Coverage   82.15%   81.96%   -0.20%     
==========================================
  Files          21       21              
  Lines        1547     1647     +100     
==========================================
+ Hits         1271     1350      +79     
- Misses        276      297      +21     
Flag Coverage Δ
unittests 81.96% <80.53%> (-0.20%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
taxbrain/report.py 86.92% <44.44%> (-3.33%) :arrow_down:
taxbrain/taxbrain.py 89.41% <82.10%> (-3.37%) :arrow_down:
taxbrain/tests/test_brain.py 94.25% <100.00%> (+0.50%) :arrow_up:
taxbrain/utils.py 92.89% <100.00%> (+0.03%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 9347165...181bed6. Read the comment docs.

andersonfrailey commented 2 years ago

Thanks for working on this, @jdebacker! Anything else you want to add before I merge?

jdebacker commented 2 years ago

Nope - this is ready - thanks!