Closed birgits closed 4 months ago
Thank you so much @AnyaHe for your valuable comments. I implemented some of your suggestions and also have a few points I would like to discuss further. See comments above :)
Thanks @birgits for addressing all the comments and the improvements in this PR. From my side, you can merge now. Great work! :)
Description
This PR mainly changes the implementation of
reduced_analysis
.reduced_analysis
can be used to conduct the grid reinforcement on a reduced number of time steps to avoid running the power flow for all time steps, including ones without grid issues, and thus speed up calculations. As it was implemented so far, thereduced_analysis
could not be used to run the reinforcement for single LV grids or a subset of time steps. The latter is needed to use it in thecatch_convergence_reinforce_grid
function to conduct grid reinforcement for the converged time steps first.The PR also now allows to weight time steps by estimated grid expansion costs analogous to how it was already possible in the selection of the most critical time intervals. This can be useful to further reduce the number of time steps to the most relevant ones.
Further, a fixes small bug in selection of time intervals.
Type of change
Checklist:
pre-commit
hooksNew and adjusted code includes type hinting nowI have made corresponding changes to the documentation