Conversation
|
|
||
|
|
||
| def get_change_table_for_zone_scaling(base_grid, resource): | ||
| n_zone = param["_".join(interconnect)]["n_zone_to_scale"] |
There was a problem hiding this comment.
This setup is saved for combinations of interconnects, such as "Western_Texas"?
There was a problem hiding this comment.
Yes. I did not set the "Western_Texas" yet in the param dict. It can be done.
|
We are checking the new plants are added correctly and the profiles are not scaled for the new plants if we have scaling by zone_id and plant_id elsewhere in the grid. Do we have checks on the profiles of the new plants other than 'not scaled'? |
No we don't. Good point, I will work on that. |
@BainanXia, commit 646b64c encloses the tests. |
BainanXia
left a comment
BainanXia
left a comment
There was a problem hiding this comment.
All tests passed. This is nice. The tests are more structural and the coverage is improved. Thanks for taking care of this tedious work.
3 participants
Purpose
Improve test coverage of the
TransformProfileclass.What is the code doing
Tests the different transformation a profile (demand and renewable) can undergo:
Where to look
The code has been completely rewritten, so everywhere
Time estimate
30 min to 1h