-
Notifications
You must be signed in to change notification settings - Fork 13
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
feat: calculate MW-miles for transmission-expanded change tables #41
Conversation
The unit tests test the individual functions with the mocks. An integration test against the real object can be run as:
|
b398e92
to
8d81f2a
Compare
I ran the following and got an error:
This is the base case scenario and hence branches are not upgraded, i.e., ct['branch'] throws you an error. I guess we could add a if or try here to handle that as you did in interstate/intrastate:
|
e6fa3a7
to
0b37531
Compare
Good catch @rouille, I added an |
d321760
to
fe4d518
Compare
fe4d518
to
e518d27
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One more
This branch adds a new script
analyze/mwmiles.py
to calculate the aggregate MW-miles of upgrades from a given Scenario object, with a correspondingtest_mwmiles.py
script. This testing functionality uses several testing tools added in the PR for carbon (#39): thetests/
folder in root, thecontext.py
file to ensure that the tests always import the right version of PostREISE, and the mock objects inmocks.py
.