search

AI4OPT / OPFGenerator

Instance generator for OPF problems
MIT License
2 stars 3 forks source link

Build sol dict #12

Closed some-wallace closed 1 year ago

some-wallace commented 1 year ago

I do not think the branch "to" active and reactive power are correct, and I do not know how to retrieve them from the model. All other values from the solution dictionary matched the solve_ac_opf values.

mtanneau commented 1 year ago

@some-wallace this PR is called "Build sol dict", but I see several other changes to the code, such as scripts to run experiments, etc... You also have merge conflicts with the main branch.

Please update your PR to only include the code that exports the solution dictionary.

mtanneau commented 1 year ago

I do not think the branch "to" active and reactive power are correct

Give me a reproducible example so I can check this locally too.

mtanneau commented 1 year ago

@some-wallace bump

some-wallace commented 1 year ago

Dual values added. Removed earlier commits and conflicts with the base branch. Apologies for the delay.

mtanneau commented 1 year ago

close in favor of #13