Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Arms slayer_st APL bug

Đang mở Phù hợp với người mới
#11,747 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 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
69/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
cpp
Lĩnh vực
game-dev

Hướng nghiên cứu

Issue nêu trực tiếp điều kiện APL Arms slayer_st. Đọc điều kiện Execute được hiển thị và kiểm tra cách diễn giải các toán tử của nó cũng như các phép kiểm tra số cộng dồn của Imminent Demise; dùng các điều kiện tương đương được đề xuất làm hướng dẫn. Công việc hoàn tất khi điều kiện khớp với các trường hợp dự kiến có 0 hoặc 2 cộng dồn, đồng thời các bước xác thực APL hoặc kiểm thử liên quan của dự án đều đạt.

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

Mô tả

Describe the bug

I think there may be a bug in the current Arms APL

actions.slayer_st+=/execute,if=buff.sudden_death.stack=2|cooldown.bladestorm.ready&(!buff.imminent_demise.stack=1|!buff.imminent_demise.stack=3)

Looks like it is intended to use Execute before a Bladestorm when Imminent Demise is not at 1 or 3 stacks — i.e. at 0 or 2 stacks. That would make sense with Unhinged, since going 0 to 1 or 2 to 3 adds an extra Unhinged Mortal

However, because ! has higher priority than =, I believe !buff.imminent_demise.stack=1 is parsed as (!buff.imminent_demise.stack)=1 rather than !(buff.imminent_demise.stack=1)

As a result, the second part of the condition effectively only succeeds at 0 Imminent Demise stacks, and the 2 to 3 case is missed

Expected behavior

If the intended logic is "Execute when Bladestorm is ready and Imminent Demise is 0 or 2", I think this should be written as either:

cooldown.bladestorm.ready&(buff.imminent_demise.stack=0|buff.imminent_demise.stack=2)

or:

cooldown.bladestorm.ready&!(buff.imminent_demise.stack=1|buff.imminent_demise.stack=3)

Ngôn ngữ chính
C++
Star
1.6k
Fork
789
Merge trung bình
8 giờ 51 phút
Pull request đã merge (30 ngày)
56

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 simulationcraft/simc

Tất cả issue của simulationcraft/simc

Issue tương tự

Thêm issue về C++

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.