Jason DeBacker

Results 152 comments of Jason DeBacker

Another thought - after step (1), you can consider implementing (2) via the [Behavioral Responses package](https://github.com/PSLmodels/Behavioral-Responses). That package may already have functions that you can leverage for updating records, etc.

@bodiyang Differences in the "c" file set that are not already documented in the "a" and "b" validation include: * `c04470`, itemized deductions above the phaseout * This variable appears...

After thinking through this with some local test failures in `test_txfunc.py` in PR #836, one idea would be the following. I was getting test failures with some of the different...

@nikhilwoodruff - thanks for this suggestion. One potential issue I see is that the Docker image can become stale and you might not find out that there are conflicts with...

Follow up. I've been able to run `pytest tests/test_module_name.py` for most modules. The only one I've noted giving an error like the above is `test_parameter_plots.py`. So perhaps issues with with...

Following up on this issue... it appears the issue is with reading files that were saved via CloudPickle in versions of Python < 3.12. I have no problems running the...

We already have separate parameter files for Python 3.11 and Python 3.9/10: ```python if sys.version_info[1] < 11: base_params = utils.safe_read_pickle( os.path.join(CUR_PATH, "test_io_data", "model_params_baseline.pkl") ) else: base_params = utils.safe_read_pickle( os.path.join( CUR_PATH,...

@talumbau PR #916 makes a change that gets the `test_SS.py` unit test passing. But not sure it's exactly what we want here.

@rickecon, @talumbau does include the else (in line) and offers an improvement over my changes in that it will use the global variable if it is not `None`.