cuopt_mcp: warn/error when cuopt_solve_lp is given a model with integer/binary variables

Đang mở
  • Có gì không ổn?
#1,942 0 bình luận 0 reaction 1 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@rg20 đang làm issue này rồi.

Từ ngày 18/9/2026.

Đánh giá

Issue này chưa được đánh giá.

Mô tả

improvement

Reported in review on #1916.

`cuopt_solve_lp`/`cuopt_solve_milp` pick a settings schema (`pdlp_settings` vs `mip_settings`) based only on which tool was called, not on the parsed model. After `Read()`, the model's variable types are known — an `.lp` file with a `Generals`/`Binaries` section submitted via `cuopt_solve_lp` silently becomes an LP relaxation (integrality dropped) instead of failing loudly.

Suggestion: in `tools.submit`, after `_read_problem`, check the model's variable types against `kind`; if `kind == "pdlp_settings"` and any variable is integer/binary, raise a clear `CuOptMCPError` telling the caller to use `cuopt_solve_milp` instead (or at minimum return a warning field in the response).

Not done in #1916 to keep that PR's scope to the review items already raised.

Ngôn ngữ chính
Cuda
Star
1.1k
Fork
255
Merge trung bình
1 ngày 22 giờ
Pull request đã merge (30 ngày)
102

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của NVIDIA/cuopt

Tất cả issue của NVIDIA/cuopt

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.