-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
560 renaming housing recovery to housing valuation recovery #565
560 renaming housing recovery to housing valuation recovery #565
Conversation
I think we'll need to add deprecated to the list of dependencies, right? |
Hmm sorry I don't follow. Could you give me an example? |
I had to install the "deprecated" library into my pip environment to run the test so we might need to add it to setup.py. Looking at the conda environment for 1.18.1, it is already there (probably part of the base conda package) or another dependency we have already pulls it in. |
Basic tests pass (checked old output vs new output), the old analysis shows a deprecation message. I also checked/built the module documentation, looks good. |
Ah i see! You are talking about the deprecated library. Yes I think I need to add it good catch! |
I added the dependencies:
|
To test: