MathUtils.h: Investigate replace/remove Clamp() function in favour of C++17 std::clamp()
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 38/100
- Type d'issue
- Refactorisation
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- cpp
- Domaine
- computer-graphics, performance
Piste de recherche
Commencez par localiser l’implémentation existante de Clamp() et tous ses sites d’appel, puis comparez-les à la référence C++17 std::clamp liée. Le travail est terminé lorsqu’il est déterminé si le remplacement préserve les performances et le comportement pour l’ordre des arguments NaN ainsi que pour +0.0 par rapport à -0.0, avec toutes les utilisations affectées identifiées.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- C++
- Étoiles
- 2.1k
- Forks
- 505
- Merge moyen
- 12 h 24 min
- PR mergées (30 j)
- 2
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de AcademySoftwareFoundation/OpenColorIO
-
Inconsistent run-time library usage for locale handling breaks GPU code generation on some systems.Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 86/100
AcademySoftwareFoundation/OpenColorIO#2351 · 2 commentaires ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
-
Build failure on GCC/MinGW (MXE): std::ifstream constructor mismatch with Platform::filenameToUTF()Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
-
ReadTheDocs - OS DeprecationOuverteDocumentation
Difficulté 1/5 Moins d'une heure Accessibilité débutants 72/100
AcademySoftwareFoundation/OpenColorIO#2280 · 1 commentaire ·
Toutes les issues de AcademySoftwareFoundation/OpenColorIO
Issues similaires
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
ROCm/FastFlowLM#887 ·
Les mainteneurs répondent en général sous 1 jour
-
bug medium priority openmp tokenizer wrong result
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
Les mainteneurs répondent en général sous 1 jour
-
good first issue invalid
Difficulté 1/5 Moins d'une heure Accessibilité débutants 72/100
DanielJerolm/gpcc#74 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
patrick-hudson/put.gd-desktop#808 ·
Les mainteneurs répondent en général sous 1 jour
-
DPI change is not fully handled in the cascading window on WindowsPeut-être pris @yukawa l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
Les mainteneurs répondent en général sous 4 jours