codecov[bot]
commented
1 year ago Codecov Report
Merging #163 (8e08409) into master (38b6e89) will decrease coverage by
0.25%. The diff coverage is57.33%.
@@ Coverage Diff @@
## master #163 +/- ##
==========================================
- Coverage 80.99% 80.73% -0.26%
==========================================
Files 95 95
Lines 6986 7060 +74
==========================================
+ Hits 5658 5700 +42
- Misses 1328 1360 +32 | Impacted Files | Coverage Δ | |
|---|---|---|
| hcipy/aperture/__init__.py | 100.00% <ø> (ø) |
|
| hcipy/util/__init__.py | 100.00% <ø> (ø) |
|
| hcipy/aperture/realistic.py | 91.11% <47.45%> (-4.61%) |
:arrow_down: |
| hcipy/util/stats.py | 96.29% <93.75%> (-3.71%) |
:arrow_down: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
I added functions to simulate the noise from the electron-multiplying gain process. This is a more accurate way to simulate the noise from EMCCD cameras. I followed the procedure from https://journals.plos.org/plosone/article?id=10.1371/journal.pone.0053671
Hirsch, Michael, et al. "A stochastic model for electron multiplication charge-coupled devices–from theory to practice." PloS one 8.1 (2013): e53671.