pybricks / pybricksdev / 21322419713

54%

push

github

pybricks.connections.pybricks: Clear halt on endpoints instead of resetting device.

Change how we ensure the USB endpoints are in a good state when connecting
to a device. Instead of resetting the device, which is not supported on all
platforms, we now clear the halt condition on both endpoints.

We were doing the reset to ensure that the even/odd data toggle state
was correct. The reset, if supported, would completely reset the
connection and re-enumerate the device, which is more that we need.
On Windows, resetting is not supported and libusb would poke all endpoints
like this as a best-effort approximation of a reset. So now, the code
reflects what is actually important and will be helpful if we switch to
something other than pyusb in the future.

75 of 265 branches covered (28.3%)

Branch coverage included in aggregate %.

2052 of 3683 relevant lines covered (55.72%)

0.56 hits per line