feat: add standalone OptSkills package - #7
Merged
Conversation
Co-authored-by: Haochen Yang <2228997793@qq.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
skills/optskills/.fujiwaranoM0kou/OptSkillsatd9e14300df4b499529c74ea1981e2c1aba0628b8.git ls-remote ... HEAD. Matching commits are silent; a changed or unavailable lookup produces at most one short note and still uses the packaged cards. This check does not fetch or inspect upstream cards.This package is independent at runtime: it does not load sibling Skills. Upstream training, clustering, agent, dataset, and evaluation systems are not included.
Validation
quick_validate.py skills/optskills: passed.python3 -m unittest discover -s tests -v: 30 tests passed with 5 existing optional OSQP/NumPy dependency skips.HEADboth returnedd9e14300df4b499529c74ea1981e2c1aba0628b8in the successful run; a later transient TLS failure also confirmed the documented non-retry, non-blocking branch..agents/skills/optskills, a fresh ephemeral Codex session discoveredoptskills, read its entrypoint, displayed the matchingassignment_problem_solverindex entry, followed that indexed path, read the card, and exited 0 without formulating or solving the example. An earlier discovery run encountered WebSocket timeouts but completed after the client fell back to HTTPS.AP_easy_1->assignment_problem_solver: CP-SATOPTIMAL, objective 84; all exact-one row and column constraints passed.MCF_easy_1->minimum_cost_network_flow: Pyomo/HiGHSok/optimal, objective 127; all 6 node balances and 12 arc-capacity checks passed.FSSP_1->flowshopscheduling: CP-SATOPTIMAL, makespan 155; duration, precedence, no-overlap, makespan, and all 6 common-permutation checks passed.These runs validate those three records only, not every example in all 103 cards. The cards remain modeling templates and may contain upstream placeholders that must be completed for a user's instance.
Review notes
git diff --checkreports them. All VeryMath-authored adapter and root files pass the focused check. This PR leaves the released card text unchanged rather than silently normalizing it.highspyavoided a dynamic-symbol conflict. This is an environment/import-order note, not a package runtime dependency.Please review before merging; this PR does not auto-merge.