[BUG] numeric_limits for f16 inherits storage-type return values
Los mantenedores suelen responder en 5 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 70/100
Línea de trabajo
Abre cpp/base/f16.hpp y lee la especialización std::numeric_limits<base::f16>: deriva de numeric_limits<storage_type> y solo sobrescribe quiet_NaN()/signaling_NaN(). Añade sobrescrituras para min(), lowest(), max(), epsilon() y round_error() que devuelvan base::f16, convirtiendo desde las implementaciones base del storage_type. Como no se reporta ningún caller, añade un static_assert o una pequeña prueba de C++ en la suite de pruebas de cpp que afirme que esos miembros devuelven base::f16; hecho significa que compila y que las aserciones pasan.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Affected code
cpp/base/f16.hpp, std::numeric_limits<base::f16<T>>
Behavior
The specialization derives from std::numeric_limits<base::f16<T>::storage_type> and overrides only quiet_NaN() and signaling_NaN() to return base::f16<T>. Other value-returning members such as min(), lowest(), max(), epsilon(), and round_error() are inherited with the storage type as their return type, rather than base::f16<T> as required by the numeric_limits<T> interface. This makes the public specialization's signatures inconsistent with its advertised type even if an implicit conversion is available at a call site.
This is based on the visible specialization; no C++ build or test was run. There is no tracked caller demonstrating the downstream compile/runtime impact.
- Lenguaje dominante
- C++
- Estrellas
- 9.2k
- Forks
- 726
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una 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 activeloopai/deeplake
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
activeloopai/deeplake#3259 ·
Los mantenedores suelen responder en 5 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
activeloopai/deeplake#3256 ·
Los mantenedores suelen responder en 5 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
activeloopai/deeplake#3254 · 1 comentario ·
Los mantenedores suelen responder en 5 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
activeloopai/deeplake#3247 · 1 comentario ·
Los mantenedores suelen responder en 5 días
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
activeloopai/deeplake#3245 ·
Los mantenedores suelen responder en 5 días
Todos los issues de activeloopai/deeplake
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
lldb
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
llvm/llvm-project#229592 · 11 comentarios ·
Los mantenedores suelen responder en 1 día
-
Bump libCEED to v1 in superbuildAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
Los mantenedores suelen responder en 3 días
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 94/100
llvm/offload-test-suite#1560 ·
Los mantenedores suelen responder en 1 día
-
cudf-polars feature request
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día