MathUtils.h: Investigate replace/remove Clamp() function in favour of C++17 std::clamp()
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 38/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- cpp
- Área
- computer-graphics, performance
Línea de trabajo
Empieza localizando la implementación existente de Clamp() y todos sus puntos de llamada, y compáralos después con la referencia enlazada de C++17 std::clamp. Se considerará terminado cuando se determine si la sustitución conserva el rendimiento y el comportamiento para el orden de los argumentos NaN y para +0.0 frente a -0.0, con todos los usos afectados identificados.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
As we are now C++17 or greater we should probably investigate replacing calls to Clamp() with std::clamp.
Care may need with both performance and behaviour of non-normals etc to ensure the behaviour remains unchanged - for example ordering of arguments when one of them is a Nan, handling of +0.0 vs -0.0 etc
https://en.cppreference.com/w/cpp/algorithm/clamp.html
At first glance our implementation appears to be directly substitutable.
- Lenguaje dominante
- C++
- Estrellas
- 2.1k
- Forks
- 502
- Merge medio
- 6 d 3 h
- PR fusionados (30 d)
- 6
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 AcademySoftwareFoundation/OpenColorIO
-
Inconsistent run-time library usage for locale handling breaks GPU code generation on some systems.Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
AcademySoftwareFoundation/OpenColorIO#2351 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
AcademySoftwareFoundation/OpenColorIO#2340 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
AcademySoftwareFoundation/OpenColorIO#2329 ·
Los mantenedores suelen responder en 1 día
-
Build failure on GCC/MinGW (MXE): std::ifstream constructor mismatch with Platform::filenameToUTF()Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
AcademySoftwareFoundation/OpenColorIO#2283 ·
Los mantenedores suelen responder en 1 día
-
ReadTheDocs - OS DeprecationAbiertoDocumentation
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
AcademySoftwareFoundation/OpenColorIO#2280 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de AcademySoftwareFoundation/OpenColorIO
Issues similares
-
chore: update release scheduleAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Los mantenedores suelen responder en 4 días
-
nth_root does not restore the rounding mode and can return an upper bound below the true rootAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
scanaislop/aislop#482 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
EchoTools/nevr-runtime#547 ·
Los mantenedores suelen responder en 1 día
-
1 type: problem 2 scope: firmware 3 good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
SpaceTeam/STS1_COBC_SW#511 ·