POCS icon indicating copy to clipboard operation
POCS copied to clipboard

Add ZWO camera bandwidthoverload arg

Open danjampro opened this issue 4 years ago • 5 comments

  • Huntsman ZWO cameras are suffering from USB timeout issues
  • Possible solution is to modify the BANDWIDTHOVERLOAD control value
  • This PR adds an argument to the ZWO camera class to allow this to be easily set from config

danjampro avatar Jan 27 '21 03:01 danjampro

Codecov Report

Merging #1071 (08b330c) into develop (cb4d26e) will increase coverage by 0.30%. The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1071      +/-   ##
===========================================
+ Coverage    84.76%   85.07%   +0.30%     
===========================================
  Files           77       78       +1     
  Lines         6531     6598      +67     
  Branches       560      560              
===========================================
+ Hits          5536     5613      +77     
+ Misses         836      827       -9     
+ Partials       159      158       -1     
Impacted Files Coverage Δ
tests/test_mount.py 100.00% <0.00%> (ø)
src/panoptes/pocs/camera/camera.py 83.44% <0.00%> (+0.46%) :arrow_up:
src/panoptes/pocs/focuser/focuser.py 76.24% <0.00%> (+1.91%) :arrow_up:
src/panoptes/pocs/dome/astrohaven.py 80.55% <0.00%> (+2.77%) :arrow_up:

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 cb4d26e...08b330c. Read the comment docs.

codecov[bot] avatar Jan 27 '21 04:01 codecov[bot]

Sorry Dan I don't have write access ;P

Physarah avatar Jan 27 '21 04:01 Physarah

@Physarah Is a property needed? Why not the solution presented in your description in https://github.com/AstroHuntsman/huntsman-pocs/issues/330?

wtgee avatar Jan 27 '21 17:01 wtgee

@Physarah Is a property needed? Why not the solution presented in your description in AstroHuntsman/huntsman-pocs#330?

@wtgee The problem is that these errors are random. We'll have to run many exposures during normal pocs operation to see if this makes a difference. This method will allow us to easily configure e.g. half the cameras on 100% and the other half on a lower value.

danjampro avatar Jan 27 '21 22:01 danjampro

@wtgee we discussed this and agreed a property is appropriate. Can we merge?

danjampro avatar Feb 09 '21 05:02 danjampro