Closed brandondube closed 4 months ago
These seem somewhat contradictory; the first quote implies Lee's hologram algorithm is present in the package, the latter pair imply it is not. I was not able to find it. All I could find is in _lib.py, the def calculate_dmd_grating function and the routines it calls. I was surprised that none of the code in _lib.py indicates which function is which algorithm. Could you please name the functions or add to their documentation which is which?
Documentation for the algorithms added: https://pyslm2.pages.dev/algorithm
The Lee hologram corresponds to the ideal
and the simple
methods.
Unrelatedly, a grammar issue: ...improved predecessors is improper English. To be a predecessor, Shih et al or Zupancic et al would have had to publish their algorithms before Lee. These could perhaps be considered "successors" but are more aptly "improved alternatives."
Fixed!
This issue is part of my JOSS review at https://github.com/openjournals/joss-reviews/issues/6315
Your paper writes,
And later,
These seem somewhat contradictory; the first quote implies Lee's hologram algorithm is present in the package, the latter pair imply it is not. I was not able to find it. All I could find is in
_lib.py
, thedef calculate_dmd_grating
function and the routines it calls. I was surprised that none of the code in_lib.py
indicates which function is which algorithm. Could you please name the functions or add to their documentation which is which?Unrelatedly, a grammar issue:
...improved predecessors
is improper English. To be a predecessor, Shih et al or Zupancic et al would have had to publish their algorithms before Lee. These could perhaps be considered "successors" but are more aptly "improved alternatives."