MathUtils.h: Investigate replace/remove Clamp() function in favour of C++17 std::clamp()
メンテナーはふだん 2 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 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
- フォーク
- 503
- 平均マージ
- 7日 16時間
- マージ済み PR（30日）
- 7
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- 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 件 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
AcademySoftwareFoundation/OpenColorIO#2340 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
AcademySoftwareFoundation/OpenColorIO#2329 ·
メンテナーはふだん 2 日以内に返信
-
Build failure on GCC/MinGW (MXE): std::ifstream constructor mismatch with Platform::filenameToUTF()オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
AcademySoftwareFoundation/OpenColorIO#2283 ·
メンテナーはふだん 2 日以内に返信
-
Documentation
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
AcademySoftwareFoundation/OpenColorIO#2280 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信
AcademySoftwareFoundation/OpenColorIO の issue をすべて見る
似ている issue
-
ai_p2
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
ClickHouse/ClickHouse#125522 ·
メンテナーはふだん 1 日以内に返信
-
[BUG] rpicam-still loops forever with "ERROR: Device timeout detected, attempting a restart!!!"オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
raspberrypi/rpicam-apps#934 ·
-
[BUG] The priority slider's keyboard-hints tooltip shows on the 'Default' label, not the sliderオープンagent:WSL bug linux LOW ui
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
メンテナーはふだん 1 日以内に返信
-
Branch-3.3.5a
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
TrinityCore/TrinityCore#32060 ·
メンテナーはふだん 3 日以内に返信
-
needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
open-telemetry/opentelemetry-cpp#4697 ·
メンテナーはふだん 1 日以内に返信