Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

asymmetry for userdata vs attributes with ShadingSystemImpl::getattribute

オープン
#1,622 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

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

調査の方向性

src/liboslexec/shadingsys.cpp の 1927 行目付近から始め、LLVM コンパイル中に公開される userdata と属性のクエリデータを比較します。属性の型、オフセット、微分について、パリティが何を意味すべきかを定義し、既存の名前を統一すべきかどうかも含めます; 完了とは、両方のクエリパスが同じ機能を提供することです。

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

説明

Problem

In https://github.com/AcademySoftwareFoundation/OpenShadingLanguage/blob/bd7fd4cf8cd4b0b75c52715737b64b204bc585bf/src/liboslexec/shadingsys.cpp#L1927 we have:

  • num_userdata
  • userdata_names
  • userdata_types
  • userdata_offsets
  • userdata_derivs

but only:

  • num_attributes_needed
  • attributes_needed
  • attribute_scopes
  • unknown_attributes_needed

which is missing a types, offsets and derivs for attributes.

This issue is to log the need for parity across these.

The mis-matching names are already a little bit confusing, so some kind of homogenization would also make sense.

Expected behavior:

I expect userdata and attributes to provide the same abilities for query during llvm compilation.

Actual behavior:

I don't have parity for userdata and attribute queries.

Steps to Reproduce

NA

Versions
  • OSL branch/version: ALL
  • OS: ALL
  • C++ compiler: ALL
  • LLVM version: ALL
  • OIIO version: ALL
NOTE

I know this should be a PR, which I'll try and get to, but for the moment, I figured an issue would make sense.

Thanks!

主要言語
C++
スター
2.3k
フォーク
414
平均マージ
4日 2時間
マージ済み PR(30日)
19

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

AcademySoftwareFoundation/OpenShadingLanguage のほかの issue

AcademySoftwareFoundation/OpenShadingLanguage の issue をすべて見る

似ている issue

C++ の issue をもっと見る

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

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