AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

Improve config cacheID algorithm to not require Yaml

オープン
#2,005 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
Feature Request
主要言語
C++
スター
2.1k
フォーク
503
PR マージ指標
30日以内にマージされた PR はありません

説明

In the nanoColor prototype, the getCacheId method on the Config class is not working properly. This is because it is generating the ID by serializing the config to Yaml and then calculating a hash of that. Since nanoColor removes the Yaml library, this no longer works. This task is to come up with an alternate mechanism for generating a Config cacheID.

The Yaml-based mechanism is arguably not ideal anyway since reordering color spaces and other trivial changes would change the ID. If people have thoughts on how they would like the Config cacheID to be calculated, please add a comment below.

Note that the Config cacheID is not used internally to the library itself, so this is something that strictly affects client code.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

nanoColor プロトタイプの Config クラスにある getCacheId メソッドから始め、現在どのように設定を Yaml にシリアライズしているかを調べます。Yaml を必要とせず、単純な並べ替えが ID にどのような影響を与えるべきかを定義する、安定した置き換え方法を決定します。クライアントコードが削除された依存関係なしで Config cacheID を計算できれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
cpp
領域
tooling
issue の種類
機能追加
難易度
5/5
見積もり時間
1週間以上
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
25/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。