MathUtils.h: Investigate replace/remove Clamp() function in favour of C++17 std::clamp()
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 新手友好度
- 38/100
- Issue 类型
- 重构
- 描述清晰度
- 基本清楚
- 活跃度
- 停滞
- 技术栈
- cpp
调研方向
首先定位现有的 Clamp() 实现及其所有调用点，然后将它们与链接的 C++17 std::clamp 参考进行比较。完成的标准是确定替换是否能保持在 NaN 参数排序以及 +0.0 与 -0.0 方面的性能和行为，并识别所有受影响的使用位置。
由索引模型根据 Issue 内容生成。
描述
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.
- 主要语言
- C++
- 星标
- 2.1k
- 派生
- 502
- 平均合并
- 6 天 3 小时
- 30 天内合并 PR
- 6
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
AcademySoftwareFoundation/OpenColorIO 的其他 Issue
-
Inconsistent run-time library usage for locale handling breaks GPU code generation on some systems.未关闭
难度 2/5 1-3 小时 新手友好度 86/100
AcademySoftwareFoundation/OpenColorIO#2351 · 2 条评论 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 84/100
AcademySoftwareFoundation/OpenColorIO#2340 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 82/100
AcademySoftwareFoundation/OpenColorIO#2329 ·
维护者通常 1 天内回复
-
Build failure on GCC/MinGW (MXE): std::ifstream constructor mismatch with Platform::filenameToUTF()未关闭
难度 2/5 1-3 小时 新手友好度 78/100
AcademySoftwareFoundation/OpenColorIO#2283 ·
维护者通常 1 天内回复
-
Documentation
难度 1/5 1 小时以内 新手友好度 72/100
AcademySoftwareFoundation/OpenColorIO#2280 · 1 条评论 ·
维护者通常 1 天内回复
查看 AcademySoftwareFoundation/OpenColorIO 的全部 Issue
相似的 Issue
-
Mac mini M6 (16 GB): Q2_0 streaming works - measured 11.7->27.2 tok/s + arena fix for sub-32 GB Macs未关闭
难度 2/5 1-3 小时 新手友好度 68/100
维护者通常 1 天内回复
-
champion good first issue remake source/qq
难度 2/5 1-3 小时 新手友好度 65/100
metasequoiaime/msime-windows#1025 ·
维护者通常 1 天内回复
-
backlog bug
难度 2/5 1-3 小时 新手友好度 75/100
EchoTools/nevr-runtime#653 ·
维护者通常 1 天内回复
-
`--fail-require-skip` never fires: run_tests.py searches stdout, but the unittest binary writes the skip summary to stderr可能已有人在做 关联的 PR 仍在进行中或已合并。 未关闭
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 75/100
InfiniTimeOrg/InfiniTime#2492 ·