-
Notifications
You must be signed in to change notification settings - Fork 266
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
Radiancemeter sensor plugin #83
Conversation
Hi @leroyvn, Thanks a lot for opening this PR. Really happy to see people porting/implementing more plugins in Mitsuba 2! The code looks great to me, and the tests as well. Just one minor comment (see above). I am curious to know if you have encountered any bugs when rendering 1x1 images with this plugin. Not sure this is something we have ever tested. |
I had similar behaviour as in #82 with both the Forcing the use of a |
Indeed it would be nice to have a warning in the constructor if the One other thing: could we add a test where we construct a simple scence (e.g. no shapes, just a |
Sure, I will do do that. |
There it is! I used numpy to check the results, is that okay? |
Yes, One minor comment regarding the filter radius check (see above) and it would be great to squash those commits into one after that. Then this should be ready to launch! 🚀 Thanks again @leroyvn |
This commit adds a radiance meter sensor. Co-authored-by: Sebastian Schunke <sebastian.schunke@rayference.eu>
This commit ports Mitsuba 1's radiance meter sensor.
Co-authored-by: @schunkes