Commit e9f6077
* Add plan for #491: ExactCoverBy3Sets model
* Implement #491: Add ExactCoverBy3Sets model
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* chore: remove plan file after implementation
* Fix Copilot review: validate distinct elements and sort triples in X3C
- Sort each triple on construction to enforce the "sorted triple" invariant
- Validate that each subset contains 3 distinct elements
- Add CLI validation before calling new() to avoid panics
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* docs: add ExactCoverBy3Sets to CLAUDE.md architecture section
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* fix(x3c): close review gaps
* fix: remove duplicate display-name entries and add missing coverage test
Remove 3 duplicate display-name entries (OptimalLinearArrangement, RuralPostman,
LongestCommonSubsequence) introduced by merge conflict in reductions.typ. Add
missing should_panic test for duplicate-element validation in X3C constructor.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
* docs: load X3C paper example from models.json, add load-model-example helper
Add load-model-example() Typst function (mirrors load-example() for rules)
that loads canonical model examples from generated models.json. Rewrite the
ExactCoverBy3Sets problem-def example to derive all concrete values from JSON
instead of hand-writing them.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
* docs: migrate all paper examples to load from JSON, add CLI X3C example
- Migrate 28 problem-def example sections in reductions.typ to use
load-model-example() instead of hand-written data. All concrete values
now derived from models.json via Typst expressions.
- Add load-model-example() helper (mirrors load-example() for rules)
- Add concrete X3C example to CLI help text and docs/src/cli.md
- 10 problems without models.json entries remain hand-written:
GraphPartitioning, OptimalLinearArrangement, BinPacking, Knapsack,
RuralPostman, SubgraphIsomorphism, LongestCommonSubsequence, SubsetSum,
MinimumFeedbackArcSet, FlowShopScheduling
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---------
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: GiggleLiu <cacate0129@gmail.com>
1 parent 5bd4da9 commit e9f6077
16 files changed
Lines changed: 1721 additions & 762 deletions
File tree
- docs
- paper
- src
- reductions
- problemreductions-cli
- src
- commands
- tests
- src
- models
- graph
- set
- unit_tests
- models/set
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
274 | 274 | | |
275 | 275 | | |
276 | 276 | | |
| 277 | + | |
277 | 278 | | |
278 | 279 | | |
279 | 280 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
89 | 89 | | |
90 | 90 | | |
91 | 91 | | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
92 | 108 | | |
93 | 109 | | |
94 | 110 | | |
| |||
0 commit comments