Fix #1095: MinimumDominatingSet/i32 to MinimumSetCovering/i32 - #1104
Fix #1095: MinimumDominatingSet/i32 to MinimumSetCovering/i32#1104isPANN wants to merge 4 commits into
Conversation
Implementation SummaryChanges
Verification
Deviations from Plan
Open Questions
|
Agentic Review ReportStructural CheckStructural Completeness
Build Status
Semantic Review
Issue ComplianceAll 6 checks pass: source/target variants, construction, extraction, correctness, overhead expressions, and the weighted-path example match issue #1095. Structural summary: 12/12 structural checks and 6/6 issue-compliance checks passed; no FAIL or ISSUE findings. Quality CheckDesign Principles
Test QualityThe suite has strong value-level coverage: exact nontrivial target structure and optimal extraction, signed weights, empty and isolated graphs, and adversarial self-loop/repeated-edge input. Important — missing explicit infeasible-configuration preservation test. The shared round-trip helper enumerates optimal target witnesses and verifies their extracted source solutions, but it does not directly evaluate a known infeasible source configuration and the identical target configuration. Add a semantic assertion such as selecting only an endpoint of Quality Summary
Agentic Feature TestsFeature:
The canonical weighted No candidate agentic finding required reproduction; there were no confirmed agentic issues. Generated by review-pipeline |
- test preservation of an infeasible endpoint-only configuration
Summary
Add the weighted closed-neighborhood reduction from Minimum Dominating Set to Minimum Set Covering.
Fixes #1095