Discussion: Should `MOI.set` perform complex operations?
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 25/100
- Loại issue
- Tài liệu
- Độ rõ ràng
- Cần làm rõ
- Mức độ hoạt động
- Ít trao đổi
- Công nghệ
- julia
- Lĩnh vực
- backend-api-design
Hướng nghiên cứu
Bắt đầu bằng cách đọc pull request 356 và các cách sử dụng hiện có của MOI.set, sau đó so sánh chúng với compute_conflict!, write_to_file, read_from_file, submit, transform và Utilities.PenaltyRelaxation. Làm rõ các ưu điểm, nhược điểm và tiền lệ của việc thực hiện các thao tác phức tạp trong set so với modify, đồng thời ghi lại các tiêu chuẩn đã thống nhất nếu cuộc thảo luận đi đến một quyết định.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
As far as I understand MOI.set is primarily used for setting attributes.
However, https://github.com/jump-dev/DiffOpt.jl/pull/356 introduced:
function MOI.set(model::Model, ::DiffOpt.ForwardDifferentiate)
as a replacement to
function forward_differentiate!(model)
Historically, performing more complex operations has been left (mostly?) for special functions such as:
compute_conflict!write_to_fileread_from_filesubmittransform
A partial counter argument is:
Utilities.PenaltyRelaxation, but it is amodifynot even aset
I do think modify here makes sense.
Are there any other arguments to each side? any other precedent?
I would like to have the reasoning for that clarified.
What are de advantages and disadvantages?
Bonus:
As a follow-up, we should have good practices and standards documented to help developers consistently make decisions in these cases.
- Ngôn ngữ chính
- Julia
- Star
- 146
- Fork
- 21
- Merge trung bình
- 1 giờ 25 phút
- Pull request đã merge (30 ngày)
- 2
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của jump-dev/DiffOpt.jl
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
jump-dev/DiffOpt.jl#362 · 1 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
jump-dev/DiffOpt.jl#348 ·
-
good first issue
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
jump-dev/DiffOpt.jl#336 · 4 bình luận ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
jump-dev/DiffOpt.jl#331 · 1 bình luận ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
jump-dev/DiffOpt.jl#318 · 1 bình luận ·
Tất cả issue của jump-dev/DiffOpt.jl
Issue tương tự
-
CI: two jobs fail on the new ubuntu-latest (Ubuntu 26.04) runner imageCó thể đã có người làm @bjarthur đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 63/100
mojaie/MolecularGraph.jl#151 ·
-
Julia runner test IDs cause false oracle regressionsCó thể đã có người làm @yu-depend đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
View of a view is brokenCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 1 ngày