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

OrenNayar with roughness=0 does not reduce to Lambertian #74

Open
6 of 8 tasks
connortbot opened this issue Sep 30, 2024 · 0 comments · May be fixed by #72
Open
6 of 8 tasks

OrenNayar with roughness=0 does not reduce to Lambertian #74

connortbot opened this issue Sep 30, 2024 · 0 comments · May be fixed by #72
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@connortbot
Copy link
Member

connortbot commented Sep 30, 2024

Branch: test (v0.1.5)

Prerequisites

  • I am using the latest available Docker container
  • I have checked that there are not duplicate existing issues
  • Documentation
  • Bug
  • Feature

Description

See expected/actual behaviour.

Steps to Reproduce

Run instances.csr or this much simpler one

# sphere.csr
version 0.1.5

Camera
lookfrom 10 0 0
lookat 0 0 0
vup 0 1 0
vfov 60
aspect_ratio 16/9
aperture 0.0001
focus_dist 10

Sky
top 0.5 0.7 1.0
bottom 1.0 1.0 1.0

Material[Diffuse]
id red
texture no
albedo 1.0 0.2 0.2
roughness 0.5

Sphere
id sphere1
position 0 -1 3
material red
radius 1
medium false

Instance[SpherePrimitive]
prim_id sphere1
translate 0 0 -6

Expected Behaviour

A correct Oren Nayar implementation should look like this:
image

Actual Behaviour

Currently even at 0 roughness the spheres look flat - as if they are always rough...
instances

The cos_theta term in the integrator has been double checked many times including the OrenNayar pdf - if either is set to use the r_in direction instead then the grazing edge darkness as seen with sigma = 0 IS evident, but then increased roughness also doesn't have the expected output. Furthermore, the horizons when oren nayar spheres are used as grounds end up ridiculously dark.

Self-Service

  • I'd be willing to fix this bug/add this feature myself
  • I'd be willing to add this documentation myself
  • I would like help/cannot do this feature/documentation
@connortbot connortbot added bug Something isn't working help wanted Extra attention is needed labels Sep 30, 2024
@connortbot connortbot linked a pull request Sep 30, 2024 that will close this issue
@connortbot connortbot removed a link to a pull request Oct 1, 2024
@connortbot connortbot linked a pull request Oct 1, 2024 that will close this issue
@connortbot connortbot reopened this Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant