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

Challenging to diagnose bad expressions when using numerical comparison operators.

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

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

Từ ngày 27/9/2025.

Đánh giá

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

Mô tả

enhancement

I have an expression where one side of a comparison operator fails (e.g. undeclared reference). The result is a "no matching overload" error, which isn't very helpful. I expect to see the original failure bubble up to the top of the evaluation.

E.g.

python3 -m celpy -n 'bad < 113.0'
ERROR: <input>:1:1 found no matching overload for 'relation_lt' applied to '(<class 'celpy.evaluation.CELEvalError'>, <class 'celpy.celtypes.DoubleType'>)'
    | bad < 113.0
    | ^

While the offending part of the expression is still in the output, it's still fairly misleading, and could cause code trying to customize error handling to do the wrong thing.

I took a quick gander where the failure occurs, and it seems to be that the builtin comparison operators don't have an overload for the CELEvalError class. Many of the others do. I'm not sure if this was a miss or on purpose.

Either way, hacking this in seemed to fix the problem:

diff --git a/src/celpy/evaluation.py b/src/celpy/evaluation.py
index 799b05c..8545a02 100644
--- a/src/celpy/evaluation.py
+++ b/src/celpy/evaluation.py
@@ -228,10 +228,22 @@ class CELEvalError(Exception):
     def __rpow__(self, other: Any) -> "CELEvalError":
         return self
 
+    def __lt__(self, other: Any) -> "CELEvalError":
+        raise self
+
+    def __le__(self, other: Any) -> "CELEvalError":
+        raise self
+
+    def __gt__(self, other: Any) -> "CELEvalError":
+        raise self
+
+    def __ge__(self, other: Any) -> "CELEvalError":
+        raise self
+
     def __eq__(self, other: Any) -> bool:
         if isinstance(other, CELEvalError):
             return self.args == other.args
-        return NotImplemented
+        raise self
 
     def __call__(self, *args: Any) -> "CELEvalError":
         return self

I can submit a PR if there's interest.

Ngôn ngữ chính
Python
Star
173
Fork
41
Không có pull request nào được merge trong 30 ngày

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

  • Có Dockerfile hoặc tệp Docker Compose
  • Không có mẫu pull request
  • Không có hướng dẫn đóng góp

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 cloud-custodian/cel-python

Tất cả issue của cloud-custodian/cel-python

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.