`box_from_file` returns `ruamel.yaml.scalarfloat.ScalarFloat` instead of `float` for floating-point values
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
Línea de trabajo
Comienza en el punto de entrada box_from_file y reproduce la MRE de YAML con el valor Float; después, sigue cómo se conservan los valores escalares de ruamel YAML. Se considera terminado cuando el valor cargado y el resultado de Box.to_dict() usan el tipo float integrado; añade o actualiza una prueba de regresión si la estructura de pruebas del repositorio identifica la ubicación adecuada.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
When I load a Box from a YAML file, the floating-pont values are represented as a type from the ruamel YAML library, and not cast to the builtin float type. This resulted in a difficult bug (as without explicitly calling type() on it, it is rendered visually as a float) where these are being improperly serialised when interacting with another library, as it doesn't know how to handle them. This happens even if I convert the box to a builtin dictionary with Box.to_dict().
MRE:
Float: 0.1
import box
float = box.box_from_file('Float.yaml')
print( float.Float )
print(type(float.Float))
outputs
0.1
<class 'ruamel.yaml.scalarfloat.ScalarFloat'>
I assume it would be more appropriate for all types from YAML/JSON/etc that can be represented as builtin to be converted as part of the process.
I would look into this further, but my debugger seems to refuse to step into the function or hit any manually-added breakpoints - I see .pyi and C wrappers, so looks like it might be implemented internally in C.
Thanks
- Lenguaje dominante
- Python
- Estrellas
- 2.8k
- Forks
- 145
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de cdgriffith/Box
-
subclass of `Box` does not unpickle correctlyPosiblemente ocupada @koriyoshi2041 la tomó hace 142 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
cdgriffith/Box#308 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 42/100
cdgriffith/Box#307 · 1 comentario ·
-
Dotted keys read from YAML are lost with box_dots / default_boxPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
cdgriffith/Box#305 ·
-
[Bug] Camel killer box and box dots gives unexpected behaviourPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 30/100
cdgriffith/Box#300 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 42/100
cdgriffith/Box#298 ·
Todos los issues de cdgriffith/Box
Issues similares
-
documentation good first issue help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
drevendev/HungryCrab#273 ·
Los mantenedores suelen responder en 1 día
-
good first issue help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
drevendev/BeautifulRepo#19 ·
Los mantenedores suelen responder en 2 días
-
Moonshot: Kimi K3 always sends `reasoning_effort="max"`, though the API now accepts `low` and `high`Abiertoaccepted
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
UKGovernmentBEIS/inspect_ai#5939 ·
Los mantenedores suelen responder en 1 día
-
bug linux
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
Fill paper for Screen2WordsAbiertoarea:data data-quality
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
reacher-z/Awesome-CUA#1468 ·
Los mantenedores suelen responder en 1 día