Forum Replies Created
-
Posts
-
作业 5，再提一个感觉最好放进框架的细节，pink room 和 box 场景因为光照数据不同需要的降噪参数不太一致，对 pink room 需要把 m_sigmaColor 调到 10 左右，其他可以保持一致。
除此之外未遇到阻碍性问题~
作业 5，一个小问题，TemporalAccumulation 函数中 alpha 默认值给了 1，导致在做 Reproject 部分时默认输出的还是 上一步的 filteredColor 而不是 m_accColor，必须要注掉 TemporalAccumulation 函数才能看到 Reproject 的输出。建议调试时先把 alpha 改为 0 就不用关心这个问题了。
作业 4，Skybox 对本次作业并没有意义，还影响观察，建议大家关掉调试：
注释掉 src/engine.js 134 行：
// loadOBJ(renderer, ‘assets/testObj/’, ‘testObj’, ‘SkyBoxMaterial’, skyBoxTransform);
src/renderers/WebGLRenderer.js 23 行开始调整一个合适的背景色：
gl.clearColor(0.1, 0.1, 0.1, 1.0); // Clear to black, fully opaque
gl.clearDepth(1.0); // Clear everything
gl.clear(gl.COLOR_BUFFER_BIT | gl.DEPTH_BUFFER_BIT);作业 4 WebGL 框架，KullaContyMaterial.js 中 uniform 名字传错了，应为 uEavgLut 而不是 uEavgFLut，这会导致 Eavg 贴图实际用的是 albedo 那张，问题比较隐蔽，但影响比较严重。
m 是半角向量，所以其实 dot(o, m) 和 dot(i, m) 是等价的，所以倒是可以互换。
WebGL 框架默认的 PBR 和 KullaConty shader 里 DistributionGGX 函数没有返回值，这个有报错，修了就画出来了。
μ 的定义感觉这里其实还是没那么严谨的，Kulla Conty 原作里面注释说 μ 就是 cos(θ)：
Notice that I’m using the Greek letter µ for the cos θ term here.
This is just to avoid too much trigonometry in the slides and
it’s closer to how the formulas are actually implemented.E_avg 我理解也应该是不需要做 sample 的，因为已经有了加权平均的过程……
作业 4 的 GeometrySchlickGGX 项代码中实际使用的是 [Karis SIGGRAPH 13] 版本的 k 值（a * a / 2），并非作业描述中提到的 (a + 1)^2 / 8，估计是为了更接近 Kulla Conty 原作所附的计算结果，两套算法不要搞混。
作业 4 描述 2.2.2 节，最后整理出的采样入射权重公式有误， 应为：(注意分子上是 dot(i, m))
weight(i) = (i · m) * G(i, o, h) / (o · n) * (m · n)
作业 4 离线预计算部分代码没有适配 MSVC，需要做以下调整：
CMake 文件加入以下段落：
if(MSVC)
target_compile_definitions(lut-Emu-MC PUBLIC _USE_MATH_DEFINES)
target_compile_definitions(lut-Eavg-MC PUBLIC _USE_MATH_DEFINES)
target_compile_definitions(lut-Emu-IS PUBLIC _USE_MATH_DEFINES)
target_compile_definitions(lut-Eavg-IS PUBLIC _USE_MATH_DEFINES)
endif()C++ 标准并未支持 VLA，需要替换两个 Eavg 实验代码中的
uint8_t data[resolution * resolution * 3];
为
std::vector<uint8_t> buffer(resolution * resolution * 3);
uint8_t* data = buffer.data();This post has received 2 votes up.作业 3 完成度明显提高，没有遇到阻碍性问题，助教同学们辛苦。
提醒一下 cave 场景的相机位置和光照信息和 cube 场景是不一致的，需要手动切换一下，不然直接用 cube 的参数进 cave 正好是墙背后，间接光照下效果是黑屏，不要被搞懵了。
这论坛限时修改还不能删帖的策略实在是有点蛋疼。。写的时候也不能预览，我不知道发出来排版竟然这么乱啊。。
格式正常的版本先放这里了：
https://docs.qq.com/doc/DUG1UZGVDamh1YXVq<h2 id=”-1″>作业 1</h2>
- <p>框架提供的 unpack 函数算法实现不准确，在不加 bias 时，会导致严重的 banding（地面一半白一半黑而不是典型的 z-fighting 效果），一定程度上影响作业调试。
准确算法参考这里：</p><code class="lang-cpp"><span class="hljs-comment">// shaders/shadowShader/shadowFragment.glsl</span> vec4 pack (<span class="hljs-type">float</span> depth) { <span class="hljs-comment">// 使用rgba 4字节共32位来存储z值,1个字节精度为1/256</span> const vec4 bitShift = vec4(<span class="hljs-number">1.0</span>, <span class="hljs-number">255.0</span>, <span class="hljs-number">65025.0</span>, <span class="hljs-number">16581375.0</span>); const vec4 bitMask = vec4(<span class="hljs-number">1.0</span>/<span class="hljs-number">255.0</span>, <span class="hljs-number">1.0</span>/<span class="hljs-number">255.0</span>, <span class="hljs-number">1.0</span>/<span class="hljs-number">255.0</span>, <span class="hljs-number">0.0</span>); <span class="hljs-comment">// gl_FragCoord:片元的坐标,fract():返回数值的小数部分</span> vec4 rgbaDepth = fract(depth * bitShift); <span class="hljs-comment">//计算每个点的z值</span> rgbaDepth -= rgbaDepth.gbaa * bitMask; <span class="hljs-comment">// Cut off the value which do not fit in 8 bits</span> return rgbaDepth; } <span class="hljs-comment">// shaders/phongShader/phongFragment.glsl</span> <span class="hljs-type">float</span> unpack(vec4 rgbaDepth) { const vec4 bitShift = vec4(<span class="hljs-number">1.0</span>, <span class="hljs-number">1</span>/<span class="hljs-number">255.0</span>, <span class="hljs-number">1</span>/<span class="hljs-number">65025.0</span>, <span class="hljs-number">1</span>/<span class="hljs-number">16581375.0</span>); return dot(rgbaDepth, bitShift); } - 课程框架的清屏操作代码不完整，实际上是没有真正每帧清屏的。glClearColor 和 glClearDepth 只是指定清屏颜色，实际的执行还得再加一句 glClear。
<code class="lang-js"><span class="hljs-comment">// renderers/WebGLRenderer.js</span> <span class="hljs-keyword">gl</span>.clearColor(0.0, 0.0, 0.0, 1.0); <span class="hljs-comment">// Clear to black, fully opaque</span> <span class="hljs-keyword">gl</span>.clearDepth(1.0); <span class="hljs-comment">// Clear everything</span> <span class="hljs-keyword">gl</span>.<span class="hljs-keyword">clear</span>(<span class="hljs-keyword">gl</span>.COLOR_BUFFER_BIT | <span class="hljs-keyword">gl</span>.DEPTH_BUFFER_BIT);
清屏操作在处理平面上的一些 artefact 时可以用上。
- 在课程视频 P3 80:26 时，闫老师的原话是："……也就意味着如果 light 离你比较远，也就是在 shadowmap 上找一个比较小的范围"。这里应该是口误，light 越远应该查询范围越大。这个细节没有在课程任何其他地方提及，除非自己翻一下 N 厂的原论文实现，考虑到不是每个同学都会这么做，所以这个口误影响比较大。
## 作业 2
* prt 框架，CMake 生成的 MSVC 项目默认对代码中的中文内容支持有问题（会吞换行符），导致编译报 lambda 没有返回值的错，解决方案是在 prt/CMakeLists.txt 112 行添加：
`cmake
target_compile_options(nori PUBLIC /utf-8) # MSVC unicode support
`
* WebGL 框架，每帧每个模型都在创建和上传相同的 vertex buffer，因而导致这回作业框架的性能相对很低，而且静置一段时间后页面还会随机崩溃。解决办法是提前 cache 这块内存，每帧只需要绑定，不需要创建和上传。
`js
// renderers/MeshRender.js insert to the back of constructor
if (material.attribs.some((a) => a === ‘aPrecomputeLT’)) {
this.#precomputeLTBuffer = gl.createBuffer();
this.#currentEnvmap = -1;
}// replace renderers/MeshRender.js:204-211
// Bind attribute mat3 – LT
if (this.#precomputeLTBuffer) {
gl.bindBuffer(gl.ARRAY_BUFFER, this.#precomputeLTBuffer);if (this.#currentEnvmap !== guiParams.envmapId) {
gl.bufferData(gl.ARRAY_BUFFER, new Float32Array(precomputeLT[guiParams.envmapId]), gl.STATIC_DRAW);
this.#currentEnvmap = guiParams.envmapId;
}for (var ii = 0; ii < 3; ++ii) {
gl.enableVertexAttribArray(this.shader.program.attribs[‘aPrecomputeLT’] + ii);
gl.vertexAttribPointer(this.shader.program.attribs[‘aPrecomputeLT’] + ii, 3, gl.FLOAT, false, 36, ii * 12);
}
}
`
* WebGL 框架，不管 shader 里是否实际用到了 mesh 里的 attribute，统一会执行 vertexAttribPoint 等操作，这在比如 shader 里没用使用 aNormalPosition 时，由于反射没取到 location 信息，接下来每帧都会产生index out of range报错。不影响运行效果，但可能对作业调试产生干扰。
* 作业描述中的效果截图都没有考虑伽马校正和积分系数 pi 的问题，所以效果其实是不正确的。可以不参考这个截图。This post has received 1 vote up.## 作业 1
* 框架提供的 unpack 函数算法实现不准确，在不加 bias 时，会导致严重的 banding（地面一半白一半黑而不是典型的 z-fighting 效果），一定程度上影响作业调试。
准确算法参考[这里](https://aras-p.info/blog/2009/07/30/encoding-floats-to-rgba-the-final/)：
`cpp
// shaders/shadowShader/shadowFragment.glsl
vec4 pack (float depth) {
// 使用rgba 4字节共32位来存储z值,1个字节精度为1/256
const vec4 bitShift = vec4(1.0, 255.0, 65025.0, 16581375.0);
const vec4 bitMask = vec4(1.0/255.0, 1.0/255.0, 1.0/255.0, 0.0);
// gl_FragCoord:片元的坐标,fract():返回数值的小数部分
vec4 rgbaDepth = fract(depth * bitShift); //计算每个点的z值
rgbaDepth -= rgbaDepth.gbaa * bitMask; // Cut off the value which do not fit in 8 bits
return rgbaDepth;
}// shaders/phongShader/phongFragment.glsl
float unpack(vec4 rgbaDepth) {
const vec4 bitShift = vec4(1.0, 1/255.0, 1/65025.0, 1/16581375.0);
return dot(rgbaDepth, bitShift);
}
`
* 课程框架的清屏操作代码不完整，实际上是没有真正每帧清屏的。glClearColor 和 glClearDepth 只是指定清屏颜色，实际的执行还得再加一句 glClear。
`js
// renderers/WebGLRenderer.js
gl.clearColor(0.0, 0.0, 0.0, 1.0); // Clear to black, fully opaque
gl.clearDepth(1.0); // Clear everything
gl.clear(gl.COLOR_BUFFER_BIT | gl.DEPTH_BUFFER_BIT);
`
清屏操作在处理平面上的一些 artefact 时可以用上。
* 在课程视频 P3 80:26 时，闫老师的原话是：”……也就意味着如果 light 离你比较远，也就是在 shadowmap 上找一个比较小的范围”。这里应该是口误，light 越远应该查询范围越大。这个细节没有在课程任何其他地方提及，除非自己翻一下 N 厂的原论文实现，考虑到不是每个同学都会这么做，所以这个口误影响比较大。看成本和具体情况的严重程度吧，实际项目里 hack 的手段可以很多。
SH 本身的输出就是线性空间，相当于光照算完了，要上屏肯定得转到 SRGB 的。
涉及 texture 也不一定需要校正（比如框架里的 skybox），是光照计算本身的空间决定是否需要校正。
课程框架做的有些问题的地方还有 envmap 其实应该用 HDR 资源的，这样才能真实还原环境信息。This post has received 1 vote up. - <p>框架提供的 unpack 函数算法实现不准确，在不加 bias 时，会导致严重的 banding（地面一半白一半黑而不是典型的 z-fighting 效果），一定程度上影响作业调试。