Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

Challenging to diagnose bad expressions when using numerical comparison operators.

Abierto
  • ¿Algo no está bien?
#123 5 comentarios 1 reacción 1 asignado Ver en GitHub

@klarose ya está trabajando en esto.

Desde el 27/9/2025.

Evaluación

Este issue todavía no se ha evaluado.

Descripción

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.

Lenguaje dominante
Python
Estrellas
174
Forks
41
Sin PR fusionados en 30 d

Preparar el entorno

  • Incluye un Dockerfile o un archivo de Docker Compose
  • Sin plantilla de pull request
  • Sin guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de cloud-custodian/cel-python

Todos los issues de cloud-custodian/cel-python

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.