Skip to content

Conversation

AdvancedImagingUTSW
Copy link
Collaborator

Tested on BT-MesoSPIM with analog only, digital only, and mixed modulation modes.

@AdvancedImagingUTSW AdvancedImagingUTSW linked an issue Oct 8, 2024 that may be closed by this pull request
Copy link

codecov bot commented Oct 8, 2024

Codecov Report

Attention: Patch coverage is 61.53846% with 25 lines in your changes missing coverage. Please review.

Project coverage is 53.86%. Comparing base (f80592e) to head (7613e84).
Report is 6 commits behind head on develop.

Files with missing lines Patch % Lines
src/navigate/model/devices/lasers/ni.py 71.15% 15 Missing ⚠️
src/navigate/model/device_startup_functions.py 16.66% 10 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1012      +/-   ##
===========================================
- Coverage    53.88%   53.86%   -0.02%     
===========================================
  Files          178      178              
  Lines        20168    20202      +34     
===========================================
+ Hits         10867    10882      +15     
- Misses        9301     9320      +19     
Flag Coverage Δ
unittests 53.86% <61.53%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

annie-xd-wang
annie-xd-wang previously approved these changes Oct 8, 2024
@AdvancedImagingUTSW AdvancedImagingUTSW merged commit a69c034 into develop Oct 8, 2024
1 check passed
@AdvancedImagingUTSW AdvancedImagingUTSW linked an issue Oct 17, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Laser Digital Only Modulation Doesn't Work

2 participants