search

PSLmodels / Behavioral-Responses

PSL module that estimates partial-equilibrium behavioral responses to tax changes simulated by Tax-Calculator
Other
5 stars 14 forks source link

Add dump option to response function and streamline tests #39

Closed martinholmer closed 5 years ago

martinholmer commented 5 years ago

This pull request adds an optional dump argument to the response function in order to resolve issue #38. As a result of this enhancement the optional trace argument of the response function is no longer useful and has been removed. Also, this pull request streamlines the tests, hopefully making them easier to understand, and breaking them into separate test functions so that the pytest distributed processing feature can execute them in parallel. There is not change in Behavioral-Response logic and the public API remains unchanged.

codecov-io commented 5 years ago

Codecov Report

Merging #39 into master will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #39   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           2      2           
  Lines         103     98    -5     
=====================================
- Hits          103     98    -5
Impacted Files Coverage Δ
behresp/behavior.py 100% <100%> (ø) :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 bb753a4...6316a06. Read the comment docs.