A number that is not finite is stored as None at a field that accepts None

Abierto Apto para principiantes
  • ¿Algo no está bien?
#140 3 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

@jeevanabhi-r ya está trabajando en esto.

Desde el 4/11/2026.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
78/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
python
Área
backend

Línea de trabajo

El issue apunta a ess/spec/request_types.py, en concreto a inline, y dice que las pruebas de validación de solicitudes se movieron a request_types_test.py. Lee primero inline y las pruebas pertinentes; después ejecuta ese archivo de pruebas. El trabajo estará terminado cuando se rechacen los números no finitos si el campo acepta None, incluso dentro de grupos y listas, y se siga cubriendo el comportamiento válido existente de None.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

bug

A number that is not finite is refused at a float field, because JSON holds it as null. At a field that also accepts None, it is stored as None and the request is accepted, so the workflow runs with None instead of the value given.

import math
from pathlib import Path
from pydantic import BaseModel
from ess.spec.request_types import held

class P(BaseModel):
    stop: float | None = None

class Q(BaseModel):
    stop: float = 1.0

files = lambda name, marker: name
local_path = lambda ref: Path('x')
held(P, 'stop', math.inf, files, local_path)  # (None, []): accepted, stored as None
held(Q, 'stop', math.inf, files, local_path)  # ParamsError: stop: has no JSON form that reads back as the value

The same holds inside groups and lists, at any field whose type accepts None.

Cause: request_types.inline checks that the stored JSON form reads back as itself (stored vs the JSON form of again), not that it reads back as the value given (made). inf becomes null in stored, null validates as None, and None has the JSON form null again, so the check passes.

Found while moving the request-validation tests to request_types_test.py.

Lenguaje dominante
Python
Estrellas
0
Forks
0
Merge medio
15 h 25 min
PR fusionados (30 d)
60

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

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 scipp/essapps

Todos los issues de scipp/essapps

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.