time.sleep behaviour for microsecond sleeps has changed in Python 3 about filament-scale-enhanced HOT 5 OPEN

Interesting, I wonder what it should be? I'd like to refactor a bunch of this to make it a bit easier to add test cases.

from filament-scale-enhanced.

I just noticed that this is trying to sleep 10 usec, so it won't behave differently in 2 vs 3. It will sleep 60+ usec on both however. It isn't really possible to sleep less than that in Python - time.sleep(0) returns "instantly" and the delay is just due to Python being not very fast.

HX711 datasheet says the minimum clock time is 0.2 usec, so just calling time.sleep(0) (or basically any Python function) should be long enough. However leaving the current sleep won't hurt, as long as you never change it to sleep while the clock is high.

Here is the relevant cpython code from 2.7:

https://github.com/python/cpython/blob/v2.7.18/Modules/timemodule.c#L1041-L1044

The number of microseconds is converted to a long, so it has to be equal or greater than 1 usec.

In modern Python it is rounded up instead of down: https://github.com/python/cpython/blob/main/Modules/timemodule.c#L2072

from filament-scale-enhanced.

I've never had to really care that much about tight timing, so that's really neat. Though I note that python can't time down to the microsecond level, at best it'll be in the realm of millisecond accuracy. And I'm guessing we're talking about the output settling time here, which has 2 values:

So we're either not waiting long enough or really not waiting long enough. It's probably largely academic at this point, but I might change it to ensure that it's actually doing what is intended.

from filament-scale-enhanced.

I hadn't considered the effect of the settling time, although that only applies when the chip is powered down. Sleeping for 50 to 400 milliseconds is not a problem though.

The tricky one is the PD_CLK timing (page 5, T3 and T4) - measured in microseconds not milliseconds.

from filament-scale-enhanced.

Yeah, likely an academic problem at this point. It works as is, so I'm inclined to leave it for now.

from filament-scale-enhanced.