Iegor Riepin

A few points: - PR does not patch documentation and release notes. - is a comment by @coroa [here](https://github.com/PyPSA/PyPSA/issues/1328#issuecomment-3190853223) addressed/resolved here? Curiosity: what is the main planned use-case here? I...

HOLY MAGIC 🤩

> Hey @Irieo, this makes sense! happy to review (hope this week). I am just wondering about the bigger picture about how operational optimization could be generalized in to `solve_network.py`...

> Looks good and conceptually I agree. There is some small optimization potential in dealing with the kwargs. > > I am against dumping example configs (potentially high-maintenance and not...

🟢

Here is excellent guide to SDDP https://optimization-online.org/2021/01/8217/ Here is a quick prototype of SDDP to solve a stochastic problem from PyPSA test suite https://iriepin.com/uploads/toy_sddp.py This method has high potential for...

There's need for a `CustomConstraint` component (or whatever name works), to streamline handling of extra constraints and their duals in post-processing. The problem is here is that dual variables are...

Two suggestions/concerns: [minor, implementation]: is there a good reason to declare aux variables in `constraints.py`? By standard variables are declared in `variables.py`, also auxiliary variables like https://github.com/PyPSA/PyPSA/blob/5a415d64ffcf287395c67b9e1a953332da2cc898/pypsa/optimization/variables.py#L239-L271 [major, conceptual]: In...

> Hi [@woody6703](https://github.com/woody6703), as far as I can see you have an issue with your setup. In line 9 your are using "n.set_snapshots(n.snapshots[::3])" without having specified the snapshots before. As...