Skip to content

atmospheric params changes#83

Merged
matthewtownson merged 5 commits into
AOtools:mainfrom
ojdf:params_changes
Jul 21, 2022
Merged

atmospheric params changes#83
matthewtownson merged 5 commits into
AOtools:mainfrom
ojdf:params_changes

Conversation

@ojdf

@ojdf ojdf commented Mar 28, 2022

Copy link
Copy Markdown
Contributor

Added an axis parameter to isoplanaticAngle and coherenceTime, making these funcions compatible with multi-dimensional arrays, with the integration performed over this axis.

Also added a rytov_variance function because it might be useful.

Had to fix some tests for atmos_conversions as well.

@codecov

codecov Bot commented Mar 28, 2022

Copy link
Copy Markdown

Codecov Report

Merging #83 (34d51b0) into main (76dd3dc) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #83      +/-   ##
==========================================
- Coverage   92.10%   92.08%   -0.02%     
==========================================
  Files          42       42              
  Lines        1988     1997       +9     
==========================================
+ Hits         1831     1839       +8     
- Misses        157      158       +1     
Impacted Files Coverage Δ
aotools/turbulence/atmos_conversions.py 100.00% <100.00%> (ø)
test/test_atmos_conversions.py 100.00% <100.00%> (ø)
aotools/functions/karhunenLoeve.py 92.13% <0.00%> (-0.38%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 76dd3dc...34d51b0. Read the comment docs.

@matthewtownson matthewtownson merged commit b3f8bfb into AOtools:main Jul 21, 2022
@ojdf ojdf deleted the params_changes branch July 26, 2022 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants