Skip to content

Commit

Permalink
Fix pen WLS absorption length typo (#35)
Browse files Browse the repository at this point in the history
* Fix pen WLS absorption length typo

This should be some orders of magnitude shorter, approx. 1/(5e5 cm^{-1})

* PEN absorption length is in mm, not cm
  • Loading branch information
ManuelHu authored Sep 9, 2023
1 parent e499b02 commit 26662a5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/legendoptics/data/pen_abslength.dat
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# nm cm
# nm mm
380.5359 0.3758
382.7277 0.4457
384.6675 0.5604
Expand Down
2 changes: 1 addition & 1 deletion src/legendoptics/pen.py
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ def pen_wls_absorption() -> tuple[Quantity, Quantity]:
.. optics-plot:: {'yscale': 'log'}
"""
wvl = np.array([70, 71, 380, 381]) * u.nm
absorp = np.array([1e3, 2e-4, 2e-4, 1e3]) * u.m # 1e3 is "infinity"
absorp = np.array([1e3, 2e-8, 2e-8, 1e3]) * u.m # 1e3 is "infinity"
assert absorp.check("[length]")
return wvl, absorp

Expand Down

0 comments on commit 26662a5

Please sign in to comment.