Comments (6)
The issue here is that the RGB values for sigma_a/sigma_s end up being represented using RGBSpectrum, which multiplies the value it returns by color space's illuminant. What we want here is RGBReflectanceSpectrum, which doesn't do that.
But, RGBReflectanceSpectrum only handles values in [0,1] (i.e., reflectances). Plus, it's clearly misnamed then.
So the proper fix will probably be to rename RGBReflectanceSpectrum to RGBSpectrum and fix it to handle values outside of [0,1], rename the old RGBSpectrum to RGBIlluminantSpectrum, and to patch everything up accordingly. This should be straightforward; I'll try to get to it soon.
from pbrt-v4.
I don't think this has to do with the conversion, for example rendering a cloudy torus with rgb values gives an interesting faint blue grid structure.
This looks similar to what surrounds the bunny, you can see some of the structure in the ears especially.
cloud.zip
Rendered with https://github.com/mmp/pbrt-v4/tree/ca0db69 on the cpu.
from pbrt-v4.
(Sorry I haven't had a chance to look into this yet, but yes, it doesn't look right..)
Possibly related, via @anderslanglands: the attached scene is a big white diffuse quad, lit by a white light. If the reflectance is 0.999, the average RGB channel values are all just under one, as would be expected
B: min 0.98291 max 1.01758 avg 0.999213 (0 infinite, 0 not-a-number)
G: min 0.994629 max 1.00098 avg 0.999048 (0 infinite, 0 not-a-number)
R: min 0.992676 max 1.00586 avg 0.998826 (0 infinite, 0 not-a-number)
If the diffuse reflectance is 1, then things go blue-ish:
B: min 0.98584 max 1.02051 avg 1.002 (0 infinite, 0 not-a-number)
G: min 0.975098 max 0.981445 avg 0.979393 (0 infinite, 0 not-a-number)
R: min 0.958984 max 0.971191 avg 0.964616 (0 infinite, 0 not-a-number)
from pbrt-v4.
This is looking good when rendered with https://github.com/mmp/pbrt-v4/tree/0e4b5ae
Edit for clarity
This is a render of the above torus scene, but with a sigma_a, sigma_s of (0.999, 0.999, 0.999) to test the majorant grid artifacts fixes.
Thanks Matt!
from pbrt-v4.
(Are you sure that one was rendered with RGB? I was still seeing a slight blue tinge with that cloudy doughnut scene that is now fixed(*) with e6b4d90. Here is ToT now:
(*) RGB of exactly 1,1,1 still comes out blue-ish; I need to chase that down but think I will file that as a separate bug. The attachment is rendered with 0.99 0.99 0.99. I've also confirmed that bunny-cloud comes out correctly with those RGB values.
Thanks for reporting this!
from pbrt-v4.
Opps sorry Matt, that was my bad I should have been more clear in what I was testing. My render in #28 (comment) was with a sigma_[as] of 0.999 as I was just testing the fix for majorant grid artifacts.
Thanks for making the Issue #54
from pbrt-v4.
Related Issues (20)
- Equation 14.24 seems missing from the online version of the pbrt-v4 book HOT 1
- Request: support light source definition via IES data HOT 1
- using --pixelstats in conjunction with --outfile do not work as expected
- Specify Volume with Transform and Camera Inside Volume in a Scene File
- PBRT hangs while rendering certain scenes with subsurface materials from a certain sample rate and higher
- Spectral range in SpectralFilm is overruled by constants in spectrum.h
- [Ubuntu] Unable to build TEV; uintptr_t not declared in scope.
- Apple M3 raytracing acceleration HOT 4
- Illegal memory was encountered when mixed textures are used.
- build pbrt in Windows with CUDA errors HOT 3
- Denoiser not being initialized for OptiX 8.0.0
- GPU compiling error with -DPBRT_FLOAT_AS_DOUBLE HOT 1
- Periodic Radiance Values for Spectral Render
- spectrum.h not defined in the pbrt/base
- gpu runs failing: Invalid PTX input HOT 1
- Build issue vs 2022 cuda 12.5 optix 7.7 win 11 HOT 2
- Final image is black when using GPU HOT 2
- GPU rendering not working with MSCV Version 14.40 and newer
- A question about CUDA constant memory initialization. HOT 1
- Trouble replicating Radiance results with diffusive materials HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from pbrt-v4.