You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
While the code does not really required normalized direction vectors, it would be good for consistency to enforce that. The way forward is probably to add this to optical element testing to identify optical elements that distort the normalization and then see if that's OK (and just re-normalize after passing through that element) or indicative of a bug.
The text was updated successfully, but these errors were encountered:
With the "propagate" function that I'm considering now, code will actually require that (or I would normalize at every point where it's required).
I think a better strategy would be to identify where it's not normalized and fix those and then add a test for test_all_optical_elements.
While the code does not really required normalized direction vectors, it would be good for consistency to enforce that. The way forward is probably to add this to optical element testing to identify optical elements that distort the normalization and then see if that's OK (and just re-normalize after passing through that element) or indicative of a bug.
The text was updated successfully, but these errors were encountered: