Fix dependencies, update doc workflow#146
Conversation
Switches macro_params.py from pandas_datareader to fredapi, fetching each series individually and concatenating into a DataFrame. Adds a clear error message with registration URL when FRED_API_KEY is unset. Updates environment.yml dependency accordingly. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #146 +/- ##
===========================================
- Coverage 45.69% 28.73% -16.96%
===========================================
Files 18 19 +1
Lines 1416 1472 +56
===========================================
- Hits 647 423 -224
- Misses 769 1049 +280
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
|
@jdebacker. I reviewed all the changes in this PR, and they look good. I am just running the example script on my local machine to make sure that works, because these are big-ish changes. Also, will you please do the following two things:
|
|
@jdebacker. In my local run of Baseline steady-state output Baseline transition path output |
|
@rickecon I had the same issue with the example script under the new baseline. I've now reverted my changes to keep the baseline the same as on |
|
@jdebacker. This looks good now. I am just running the example script locally to make sure it works. |
|
@jdebacker. I ran this locally, and everything solved smoothly: Baseline steady-state equilibrium solution Baseline transition path equilibrium solution (34 min, 8.9 sec) Reform steady-state equilibrium solution Reform transition path equilibrium solution (36 min, 34.9 sec) |
|
I created a new conda environment and ran |
|
@jdebacker. I've fixed the documentation issues in a PR to your branch. But I am also having two local tests fail in |
Update documentation and some cosmetics
|
@jdebacker. This ended up being a hefty PR. Thanks for getting it moving. I am merging now. |
Updated version date and added new features and fixes.
This PR does the following:
pandas-datareaderpackage with thefredapipackage inmacro_params.py.