Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add renderer presets #200

Open
luisbarrancos opened this issue Mar 10, 2019 · 6 comments
Open

Add renderer presets #200

luisbarrancos opened this issue Mar 10, 2019 · 6 comments

Comments

@luisbarrancos
Copy link
Member

Quick settings for Draft, Intermediate, Intermediate Denoised, Production.

@luisbarrancos luisbarrancos added this to the 1.2 milestone Mar 10, 2019
@luisbarrancos luisbarrancos self-assigned this Mar 10, 2019
@luisbarrancos
Copy link
Member Author

Initial test here, settings up for review and discussion later (i.e, define production settings): 4addda3

@dictoon
Copy link
Member

dictoon commented Mar 10, 2019

Settings presets are a great idea.

@luisbarrancos
Copy link
Member Author

luisbarrancos commented Mar 24, 2019

Attached screenshots for

  • Draft
  • Intermediate
  • Production
  • Intermediate Denoise

as_presets

Missing Production Denoise, but it would be best first to get some consensus on what we define as draft, intermediate and production quality settings.

Draft
as_draft_preset

Intermediate
as_intermediate_preset

Production
as_production_preset

Intermediate Denoise
as_intermediate_denoise_preset

@luisbarrancos
Copy link
Member Author

Once defined i suppose we would need to add equivalent presets for appleseed-max|blender|studio, gaffer (? @est77 )

@dictoon
Copy link
Member

dictoon commented Apr 2, 2019

Nice!

A few remarks:

  • I would always disable caustics by default, in all presets.
  • I would set the Max Global Bounces to 8, even in the Production preset.
  • I would reduce the number of Global Bounces in the Draft preset, 3 is probably enough.
  • I'm not 100% sure that Min Samples = 1 is a reasonable value, will the adaptive tile renderer be able to compute an initial noise estimate? Ping @oktomus.

As for the other settings (number of direct and IBL rays, etc.), I trust your experiments and experience here, you certainly know better than me which settings work best.

@luisbarrancos
Copy link
Member Author

@dictoon got it. I'll set the min samples to 4 for the preview mode.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Live Roadmap
Awaiting triage
Development

No branches or pull requests

2 participants