search

sot / kadi

Chandra commands and events
https://sot.github.io/kadi
BSD 3-Clause "New" or "Revised" License
5 stars 3 forks source link

WSVIDALLDN should set ccd_count = 0 #122

Closed jzuhone closed 6 years ago

jzuhone commented 6 years ago

The ACIS command WSVIDALLDN powers down all the video boards, and as such should set ccd_count = 0.

This is essentially identical to what is implemented in sot/cmd_states#39.

taldcroft commented 6 years ago

In this case it should be straightforward to add a test. Code looks fine.

taldcroft commented 6 years ago

Ping @jzuhone - for the record, waiting for either a new unit test or at least a functional description of local testing that demonstrates this works.

jzuhone commented 6 years ago

@taldcroft right--sorry. got sidetracked by some other stuff, will get back to this soon hopefully.

jzuhone commented 6 years ago

@taldcroft I moved the test

jzuhone commented 6 years ago

@taldcroft too late to squeeze these in for LRB approval?