MathUtils.h: Investigate replace/remove Clamp() function in favour of C++17 std::clamp()
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 38/100
- Issue-Typ
- Refactoring
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- cpp
- Bereich
- computer-graphics, performance
Rechercherichtung
Beginne damit, die bestehende Clamp()-Implementierung und alle ihre Aufrufstellen zu finden, und vergleiche sie anschließend mit der verlinkten C++17 std::clamp-Referenz. Als abgeschlossen gilt die Aufgabe, wenn festgestellt wurde, ob die Ersetzung die Leistung und das Verhalten bei der Reihenfolge von NaN-Argumenten sowie bei +0.0 gegenüber -0.0 beibehält, wobei alle betroffenen Verwendungen identifiziert werden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- C++
- Sterne
- 2.1k
- Forks
- 502
- Ø Merge
- 6 T. 3 Std.
- Gemergte PRs (30 T.)
- 6
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus AcademySoftwareFoundation/OpenColorIO
-
Inconsistent run-time library usage for locale handling breaks GPU code generation on some systems.Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 86/100
AcademySoftwareFoundation/OpenColorIO#2351 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
AcademySoftwareFoundation/OpenColorIO#2340 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
AcademySoftwareFoundation/OpenColorIO#2329 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Build failure on GCC/MinGW (MXE): std::ifstream constructor mismatch with Platform::filenameToUTF()Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
AcademySoftwareFoundation/OpenColorIO#2283 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Documentation
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 72/100
AcademySoftwareFoundation/OpenColorIO#2280 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in AcademySoftwareFoundation/OpenColorIO
Ähnliche Issues
-
agent:Windows bug code-review LOW testing
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 63/100
Maintainer antworten meist innerhalb von 1 Tag
-
[Cleanup] Include the public header in work_split.cppEvtl. vergeben @blozano-tt hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
tenstorrent/tt-metal#60850 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Build: the CUDA backend fails without cuDNN instead of skipping the cuDNN shim (as the cuDF shim does)Evtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
beehive-lab/TornadoVM#1248 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
SimVascular/svZeroDSolver#253 ·
-
core: RAC_LLM_OPTIONS_DEFAULT.min_p disagrees with the IDL's annotated defaultEvtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offenbug core p3
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
RunanywhereAI/runanywhere-sdks#1083 ·
Maintainer antworten meist innerhalb von 5 Tagen