Closed gschlafly closed 4 months ago
With the assert statements added in commit 2e75ba82b8944b95087f2f26c11c54e95c27b7ea to the estimated-vol branch to check that the optic axis vector do not have norm zero, 9 of the 42 tests fail.
Similar issue #61:
If the optic axis is not a unit vector, the projection of the ray onto the optic axis is misleading. In e4a27d7, we normalized the optic axis in the projection. The same should be done for the torch process.
Optic axis is normalized after each iteration update.
Description
When performing an iterative reconstruction, the optic axis estimates do not remain of unit length. This could be okay if only the direction of the vector was relevant in the forward model. However, the magnitude indirectly plays a role in the Jones calculus computations.
Files
https://github.com/PolarizedLightFieldMicroscopy/forward-model/blob/095e3eb406add7872218c7b549d77e4e6a2b72f5/VolumeRaytraceLFM/birefringence_implementations.py#L1102-L1106
To Reproduce
Functions extracted from
birefringence_implementations.py
:The following example produces an incorrect retardance and azimuth angle.
Solution ideas