Closed trevorb1 closed 1 month ago
@ktehranchi, are you able to take a quick look (and maybe a quick run) to make sure this looks okay, please. Just want to make sure this stays true to your multi-horizon implementation implementation.
Ive run a few different edge cases, and everything seems to work correctly! Comparing against the develop branch, the objective cost continually comes out ~4% different; however the results seem the same (just visually comparing graphs). 4% seems like a lot for simply numerical errors, tho. I can do a better exploration to see where the differences are coming from if you think this is needed.
Closes #335
Changes proposed in this Pull Request
In this PR I have changed the indexing logic to be the planning horizon instead of the profile year. This includes:
build_base_network
expand_demand
function is a little awkward, but it will allow for addition of scaling demand relatively easily.add_electricity
to account for planning horizon (ie. renewables, fuel prices, ect...) through updatedbroadcast_investment_horizon_index
functionI have checked that both single horizon and multi-horizon data still works. Todo before merging:
Checklist
envs/environment.yaml
.config.default.yaml
.doc/configtables/*.csv
.