imSim
imSim copied to clipboard
Consistent application of vignetting with and without using RubinOptics
At #368 , Josh asks:
- Should we require
input.vignettingif using aRubinOpticsphoton op?- What happens if
input.vignettingis populated and we're photon-shooting withoutRubinOptics? are the FFT and shoot objects still consistent in this case (i.e., does the vignetting affect the non-RubinOpticsshoot objects?)
For the first question, Mike recommends we emit a warning if input.vignetting isn't populated. For the second, it seems we should apply the vignetting scaling to the non-RubinOptics shoot objects.
Jim, is this something that can wait until after 2.0? Or do we need to address it right away?
If we don't plan to run with RubinOptics disabled for any serious work (i.e., case 2 above), then I would say this can wait. Nevertheless, I may find time to work on it in the next week or so.