AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

Add .none(false) for arguments of python exposed functions expecting strings

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

説明

Several API functions are taking const char * parameters. They are often exposed to Python with:
.def("function", &Class::function, "parameter"_a)
This is working fine, when used in python, the parameter type is checked, but when used with None a segfault occurs.
To handle that .none(false) can be added to the parameter:
.def("function", &Class::function, "parameter"_a.none(false))

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

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

調査の方向性

ファイルもテストも指定されていません。まず、const char* パラメータを持つ関数に対する .def 呼び出しを含む C++ の Python 公開用 bindings を探し、次に None のケースを再現して影響を受ける API を特定します。これらの公開された文字列パラメータが segfault なしに None を拒否し、影響を受ける bindings のカバレッジがある状態になれば完了です。

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

評価

技術スタック
cpp, python
領域
api
issue の種類
バグ
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
38/100

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

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