What is the best way to match against optional attributes?

Đang mở
#1,902 2 bình luận 0 reaction 0 người được giao Xem trên GitHub

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

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
30/100
Loại issue
Tính năng
Độ rõ ràng
Cần làm rõ
Mức độ hoạt động
Đình trệ
Công nghệ
python
Lĩnh vực
compilers

Hướng nghiên cứu

Issue không nêu tên tệp, bài kiểm thử hoặc điểm vào nào. Hãy bắt đầu bằng cách xác định API so khớp mẫu được sử dụng cho DFT và so sánh API đó với bốn trường hợp thuộc tính tùy chọn được liệt kê; để hoàn thành, cần quyết định một cách tiếp cận nhằm bao phủ các trường hợp đó mà không tạo ra các quy tắc trùng lặp không cần thiết.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

category: question module: rewriter

Suppose we need to remove an optional input of a node, for example, we want to replace DFT(x, None, -2) with DFT(x), since the default axis is -2, so we can remove the axis input.

The DFT node has two optional attributes (onesided and inverse). What is the best way to write pattern to cover these cases:

  • DFT(x, None, -2) -> DFT(x)
  • DFT(x, None, -2, onesided=1) -> DFT(x, onesided=1)
  • DFT(x, None, -2, inverse=1) -> DFT(x, inverse=1)
  • DFT(x, None, -2, inverse=1, onesided=1) -> DFT(x, inverse=1, onesided=1)

It seems we need write four rules for the four cases, is there any better solutions? cc @justinchuby

Ngôn ngữ chính
Python
Star
457
Fork
149
Merge trung bình
3 ngày 20 giờ
Pull request đã merge (30 ngày)
22

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 microsoft/onnxscript

Tất cả issue của microsoft/onnxscript

Issue tương tự

Thêm issue về Python

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.