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

Radiancemeter sensor plugin #83

Merged
merged 1 commit into from
Apr 16, 2020

Conversation

leroyvn
Copy link
Contributor

@leroyvn leroyvn commented Apr 14, 2020

This commit ports Mitsuba 1's radiance meter sensor.

Co-authored-by: @schunkes

@leroyvn leroyvn changed the title Added radiancemeter plugin Radiancemeter sensor plugin Apr 14, 2020
@Speierers
Copy link
Member

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.

@leroyvn
Copy link
Contributor Author

leroyvn commented Apr 14, 2020

I had similar behaviour as in #82 with both the perspective and radiancemeter plugins, but I don't think it's due to the film size.

Forcing the use of a box reconstruction filter could be good. Maybe I should change something to implement this behaviour?

@Speierers
Copy link
Member

Forcing the use of a box reconstruction filter could be good. Maybe I should change something to implement this behaviour?

Indeed it would be nice to have a warning in the constructor if the film->reconstruction_filter() isn't a box filter. Or maybe it is enough to check whether the radius of that filter is <= 0.5f?

One other thing: could we add a test where we construct a simple scence (e.g. no shapes, just a constant emitter) with this sensor, and call scene.integrator().render()?

@leroyvn
Copy link
Contributor Author

leroyvn commented Apr 15, 2020

Sure, I will do do that.

@leroyvn
Copy link
Contributor Author

leroyvn commented Apr 15, 2020

There it is! I used numpy to check the results, is that okay?

@Speierers
Copy link
Member

Speierers commented Apr 16, 2020

There it is! I used numpy to check the results, is that okay?

Yes, numpy is fine when dealing with Bitmap.

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>
@Speierers Speierers merged commit 8d5b37d into mitsuba-renderer:master Apr 16, 2020
@leroyvn leroyvn deleted the radiancemeter branch April 17, 2020 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants