Matt Liberty
Matt Liberty
> I have started a secure-CI for this changes. Any update?
It's not obvious enough so I'll need a test case.
The code changes are fine but you need to update the tests.
save_flow_metrics save_flow_metrics_limits
This means this PR will need a full CI testing as it may change results on all ORFS designs. Let me know when it is ready for that.
bad_vertices_ needs to be cleared at the end of recoverPower so it doesn't persist between calls.
Ready for review?
I tested the current recover power designs and they look fine. It just need the clear mentioned above to be ready to merge.
The asap7 repo is named this way so it is at least consistent. @tspyrou