サンプルコードにも間違いはある?

MayaのDGNodeを使ったShaderの書き方をサンプルコード読みながら勉強してたんだけど、そもそも参考にしてたサンプルコード自体に間違いがあることに気づいた。
前にも貼ったPhongShaderのサンプル
サンプルをそのままビルドして実行してみればわかるんだけど、specular処理がちょっとおかしなことになる。

これはごく初歩的なミスで、要するに値の丸め忘れ。
サンプルコードだと、compute関数の中でif文使って( blindDataがNullな時とそうでない場合を)振り分けた後に、ほぼ同じShading処理を二箇所に書く構造になっていて、その内の一箇所に丸め忘れがある。
不思議なことにコードを見てると、二箇所のShading処理で別々の人が書いたように書き方が変わってたりする。(コメントの書き方も何か違う)

それで思ったんだけど、そもそもこのコードの書き方がミスを犯しやすい構造になってる気がする。if文書く位置をちょっと変えるだけでもっとメンテナンスしやすいのに…(この方が実行速度が速いんだろうか)
とか書いてたらまたおかしいところ見つけたぞ。(まあ良いか)
たぶん、最初に誰かが書いたのをバージョンアップか何かで他の誰かが書き換えて行くうちにおかしなことになっちゃったんだろう。

そんで、コード直すついでにDiffuseに関してはこの考えに従って書き換えてみた。(デザイン系ではあまり考えなくて良い部分ではある)



さて、CookTorranceがうまくいかないわけだが…。

関連記事

リアルタイム レイトレーシングAPI『DirectX Ray...

ZBrushでリメッシュとディティールの転送

SDカードサイズのコンピューター『Intel Edison』

Math.NET Numerics:Unityで使える数値計...

TensorSpace.js:ニューラルネットワークの構造を...

bpy-renderer:レンダリング用Pythonパッケー...

ZBrushで仮面ライダー3号を造る 仮面編 ClipCur...

Raspberry Pi 2を買いました

ポリゴン用各種イテレータと関数セット

Geogram:C++の3D幾何アルゴリズムライブラリ

Mitsuba 2:オープンソースの物理ベースレンダラ

リメイク版ロボコップスーツのメイキング

CGレンダラ研究開発のためのフレームワーク『Lightmet...

WinSCP

写真から3Dメッシュの生成・編集ができる無料ツール『Auto...

Texturing & Modeling A Pro...

ラクガキの立体化 分割ラインの変更・バランス調整

デザインのリファイン再び

Raspberry Pi

PyDataTokyo主催のDeep Learning勉強会

3D復元技術の情報リンク集

書籍『ゼロから作るDeep Learning』で自分なりに学...

Mayaでリアルな布の質感を作るチュートリアル

Shader.jp:リアルタイム3DCG技術をあつかうサイト

OpenCV バージョン4がリリースされた!

OpenCV 3.3.0-RCでsfmモジュールをビルド

Model View Controller

iOSデバイスと接続して連携するガジェットの開発方法

PeopleSansPeople:機械学習用の人物データをU...

CGWORLD CHANNEL 第21回ニコ生配信は『シン・...

CGALDotNet:計算幾何学ライブラリ CGALのC#ラ...

DUSt3R:3Dコンピュータービジョンの基盤モデル

Caffe:読みやすくて高速なディープラーニングのフレームワ...

ZBrushのTranspose Masterでポーズを付け...

Blenderでよく使うaddon

このブログのデザインに飽きてきた

Unreal Engine Tokyo MeetUp!へ行っ...

注文してた本が届いた

PolyPaint

オープンソースの人体モデリングツール『MakeHuman』の...

BlenderのRigifyでリギング

2D→3D復元技術で使われる用語まとめ

コメント