-
Notifications
You must be signed in to change notification settings - Fork 173
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
Improve equivalent transformation calculations in base.nc #978
Conversation
As a comment, there is also an effect of E.g. for Al-steel wires Obviously, that is a purely data-based perspective ideally should can be adjusted according to the real-world operation of the existing power grids. |
A quick demonstration of the effect of the linetype definition of the transmision capacity using the notebook by @GbotemiB (thanks for the handy tool!):
|
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.
Thanks katia!
It seems this PR is ready, but could you please explain the image you posted?
I don't see how much we improve or so.
I believe it is also worth adding a release_note in this case
Thank you, Davide :) The image was in fact rather a problem statement, not testing results. The point is, the PR is dealing with a part of uncertainty which relates to the voltage mapping, while we need some other approach to deal with the uncertainty linked with definition of the line type. The picture above demonstrates this effect of the linetype. (Sorry for messing things up!) Testing results for the PR itself are as follows. When looking into data for Norway:
So, we are still two times off, but I'd say that is quite an improvement :D The topology comparison looks like that: |
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.
Many thanks @ekatef and totally understand!
Then, I think this PR can be merged as-is, that's great!
A suggestion on a possible to partially address #862 and #887 to decrease errors introduced by
base.nc
.Changes proposed in this Pull Request
Currently, we are using voltage rebase in
base.nc
which purpose is to map all the voltages to a few "standard" values. However, information on the original voltage is lost during this rebase transformation.The suggestion is:
num_parallel
for the lines after the voltage rebase to compensate an effect of the voltage increase on the transmission capacity introduced during rebase.Checklist
envs/environment.yaml
anddoc/requirements.txt
.config.default.yaml
andconfig.tutorial.yaml
.test/
(note tests are changing the config.tutorial.yaml)doc/configtables/*.csv
and line references are adjusted indoc/configuration.rst
anddoc/tutorial.rst
.doc/release_notes.rst
is amended in the format of previous release notes, including reference to the requested PR.