MathUtils.h: Investigate replace/remove Clamp() function in favour of C++17 std::clamp()
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 38/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- cpp
- Lĩnh vực
- computer-graphics, performance
Hướng nghiên cứu
Bắt đầu bằng cách tìm implementation hiện có của Clamp() và tất cả các vị trí gọi nó, sau đó so sánh chúng với tài liệu tham chiếu C++17 std::clamp được liên kết. Hoàn tất nghĩa là xác định liệu việc thay thế có giữ nguyên hiệu năng và hành vi đối với thứ tự các đối số NaN cũng như +0.0 so với -0.0 hay không, đồng thời xác định tất cả các cách sử dụng bị ảnh hưởng.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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.
- Ngôn ngữ chính
- C++
- Star
- 2.1k
- Fork
- 502
- Merge trung bình
- 6 ngày 3 giờ
- Pull request đã merge (30 ngày)
- 6
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của AcademySoftwareFoundation/OpenColorIO
-
Inconsistent run-time library usage for locale handling breaks GPU code generation on some systems.Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
AcademySoftwareFoundation/OpenColorIO#2351 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
AcademySoftwareFoundation/OpenColorIO#2340 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
AcademySoftwareFoundation/OpenColorIO#2329 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Build failure on GCC/MinGW (MXE): std::ifstream constructor mismatch with Platform::filenameToUTF()Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
AcademySoftwareFoundation/OpenColorIO#2283 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
ReadTheDocs - OS DeprecationĐang mởDocumentation
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 72/100
AcademySoftwareFoundation/OpenColorIO#2280 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của AcademySoftwareFoundation/OpenColorIO
Issue tương tự
-
chore: update release scheduleĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Maintainer thường phản hồi trong vòng 4 ngày
-
nth_root does not restore the rounding mode and can return an upper bound below the true rootĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
scanaislop/aislop#482 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
EchoTools/nevr-runtime#547 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
1 type: problem 2 scope: firmware 3 good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
SpaceTeam/STS1_COBC_SW#511 ·