SPPM renderer (CPU-only)

[Chawl]

Tx guibou. Here is a quick demo of the short solution. Direct lighting bursts contrast and effects noise distribution as expected, and "0" has issues with some reflections (speculars?), but if handled with care, this method may indeed be usable.

Camera tonemapper is Autolinear, clamping is Preserve Luminosity, and filter is Gaussian for all shots.

SPPM + Direct Lighting, Glossy threshold=0

MLT (unbiased preset)

Direct Lighting only

[neo2068]

On my system (windows 7 x64) luxrender crashes with sppm and direct lighting enabled. I tested the latest version (4dcc7b73704b) and the weekly build from 04.03.12, too.

[J the Ninja]

On my system (windows 7 x64) luxrender crashes with sppm and direct lighting enabled. I tested the latest version (4dcc7b73704b) and the weekly build from 04.03.12, too.

You have to select "random" as your sampler with direct light sampling, or else it will crash. No clue why, but that's what happens.

[neo2068]

On my system (windows 7 x64) luxrender crashes with sppm and direct lighting enabled. I tested the latest version (4dcc7b73704b) and the weekly build from 04.03.12, too.

You have to select "random" as your sampler with direct light sampling, or else it will crash. No clue why, but that's what happens.

Thanks, that works.

[guibou]

On my system (windows 7 x64) luxrender crashes with sppm and direct lighting enabled. I tested the latest version (4dcc7b73704b) and the weekly build from 04.03.12, too.

You have to select "random" as your sampler with direct light sampling, or else it will crash. No clue why, but that's what happens.

One day I'll have to hard code the fact that a sampler other than random cannot be used with SPPM...

[guibou]

I pushed some commit which can be of interest

Code: Select all

changeset:   3615:dd2b744ae73d
summary:     SPPM: fix direct light sampling with passthrough material

changeset:   3617:193c4e110120
summary:     SPPM: fix direct light sampling issue with arealigh


Those are bugfixes of direct lighting. Normally directlighting in SPPM should now behave correctly (not creating light if there is architectural glass or LOT of fireflies removed)... (BinaryCortex: you where right, I was wrong, SPPM was bugged, I'm ashamed

Code: Select all

changeset:   3616:d6313258d504
summary:     SPPM: remove the glossy threshold behavior

changeset:   3619:5209b387c31b
summary:     SPPM: add one parameter *storeglossy*


Ok, you will kill me. I removed the glossythreshold. It is bugged, not mathematically correct, and bugged. Normally with default settings SPPM is able to render anything correctly (except if there is path with only glossy surfaces and point light).

To improve the rendering of glossy surface you can:

- enable direct lighting in SPPM.
- Use glossyCoating with matte base material, it should work like the previous glossythreshold, but without bugs

For those who still have issues with glossy, or good reasons, I added a setting "storeglossy" which will store hitpoints on glossy. Use it with caution because It may get a lot of ugliness in your renders, but it will gives you correct result (in more time)

Code: Select all

changeset:   3620:aa00a3f165cc
summary:     SPPM: change default values


I changed two default behavior in SPPM:

- direct lighting is now ON by default (Any reason to switch it off is a bug and should be reported so we can fix it)
- PPM proba is now ON by default (I'm planning of removing the old heuristic soon, so please, give feedbacks)

[Chawl]

Whoa This is really a milestone

Tx for all guibou, so let's binarize some code.

Yet VS2008 doesn't like a declaration and whines a bit:

Code: Select all

1>c:\***\lux\renderers\sppm\hitpoints.h(36) : warning C4099: 'lux::PhotonData' : type name first seen using 'class' now seen using 'struct'
1>        c:\***\lux\renderers\sppm\lookupaccel.h(39) : see declaration of 'lux::PhotonData'

[Chawl]

Newest Luxblend (68aae6e43d40) is broken, "storeglossy" parameter should not be float but boolean.
For now, you can use my corrected one by the way.

[J the Ninja]

Newest luxblend (68aae6e43d40) is broken, "storeglossy" parameter should not be float but boolean.
For now, you can use my corrected one by the way.

EDIT: Oops, seems I got a bit lazy with the find and replace, forgot to fix it in the export block, official version should be fixed now.

[Chawl]

Wow, lightening fast
Tx.