adafruit / circuitpython_nau7802 Goto Github PK

I keep getting the following error using the NAU7802 with this library on RPi 5 and Python 3.11.2. This error occurs on killing the simple test in the examples directory. When I extend this code for my own application the same error occurs each time the __pu_cycle_ready bits are read during a call to nau7803.available.

Traceback (most recent call last):
  File "/home/pi/Documents/unfinished arch/nau_test.py", line 66, in <module>
    value = read_raw_value()
            ^^^^^^^^^^^^^^^^
  File "/home/pi/Documents/unfinished arch/nau_test.py", line 39, in read_raw_value
    while not nau7802.available():
              ^^^^^^^^^^^^^^^^^^^
  File "/home/pi/Documents/unfinished arch/lib/python3.11/site-packages/cedargrove_nau7802.py", line 297, in available
    return self._pu_cycle_ready
           ^^^^^^^^^^^^^^^^^^^^
  File "/home/pi/Documents/unfinished arch/lib/python3.11/site-packages/adafruit_register/i2c_bit.py", line 59, in __get__
    i2c.write_then_readinto(self.buffer, self.buffer, out_end=1, in_start=1)
  File "/home/pi/Documents/unfinished arch/lib/python3.11/site-packages/adafruit_bus_device/i2c_device.py", line 140, in write_then_readinto
    self.i2c.writeto_then_readfrom(
  File "/home/pi/Documents/unfinished arch/lib/python3.11/site-packages/busio.py", line 224, in writeto_then_readfrom
    return self._i2c.writeto_then_readfrom(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/pi/Documents/unfinished arch/lib/python3.11/site-packages/adafruit_blinka/microcontroller/generic_linux/i2c.py", line 98, in writeto_then_readfrom
    readin = self._i2c_bus.read_i2c_block_data(
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/pi/Documents/unfinished arch/lib/python3.11/site-packages/Adafruit_PureIO/smbus.py", line 264, in read_i2c_block_data
    ioctl(self._device.fileno(), I2C_RDWR, request)
KeyboardInterrupt

Instead of passing integers, which are then looked up as constants with a particular name, it would be better to pass in the actual constants, and type-check them as necessary.

If we are going to maintain this, it should be renamed to Adafruit_CircuitPython_NAU7802.

@kattni will need CI/PyPI and all that good stuff - just a reminder so i can eject from my brain

i think this line is returning true because it is a bound method, not because the ADC is actually ready for another call

https://github.com/CedarGroveStudios/CircuitPython_NAU7802/blob/154d2192e5b8cb0e55fa0d86cfb882477723637a/examples/nau7802_simpletest.py#L42

Please could you publish this in PyPi so it can be installed, with dependencies, using PIP3?
Ref: https://forums.adafruit.com/viewtopic.php?p=971026#p971026

There are a number of places in this library that use time.sleep(...) during various initialization steps. Since these are blocking they prevent any additional work, so for example if you want to initialize 10 of these devices from a single controller, you will need to wait 7.5 seconds in total.

It would be relatively easy to implement an async version that depends on asyncio which would allow non-blocking sleep configuration. I don't know if there's a nice way to do this to avoid adding the async dependency for people not using it. Is it best to just create a separate library?

Would someone please write an example that changes the poll rate to a higher speed? I'm having trouble making it work. The only thing that has worked for me is to change the default in my local copy of cedargrove_nau7802.py.
Thanks!

Re this thread for example with Gemma M0:
https://forums.adafruit.com/viewtopic.php?p=992444