powsybl / powsybl-open-rao

A toolbox based on powsybl framework dedicated to power systems coordinated capacity calculation and security analysis projects
Mozilla Public License 2.0
15 stars 6 forks source link

Improve MIP code #1049

Open pet-mit opened 3 months ago

pet-mit commented 3 months ago

Describe the current behavior

In https://github.com/powsybl/powsybl-open-rao/pull/1014, we had to do a workaround in order to fix a bug, by filtering NaN values from the MIP. In order to go fast, we had to replace the "updateBetweenSensiIterations" content by a reset of the matrix, followed by a "fill" and an old "update". The "update" is only needed because its signature allows passing more info to the matrix than "fill".

Describe the expected behavior

Make the "fill" methods more complete, and remove the need of the "updateBetweenSensiIterations" methods in the fillers. This will make the code more simple.

Describe the motivation

Cleaner code

Extra Information

No response