TypeError

[pimoroni]

TypeError

[Gadgetoid]

I can't remember the exact motivation behind this eldritch horror, but I don't believe I use device.register.set_field() anywhere in any library based on i2c-device.

Presumably I wanted device.some_settings_register.set_field(1) to work, but I seem to exclusively use the pattern:

device.set("REGISTER", key=value, key2=value2)

I would be delighted to remove this ghastly abomination and patch any libraries that show otherwise. Sometimes simpler is better, and trying to do weird dictionary lookup catching nonsense liks this is just a result of nobody telling me when to stop.

[Thynix]

I'm running into this too - maybe the __dict__ signature changed or something? I haven't looked into it yet. _RegisterProxy explains why it's there:

class _RegisterProxy(object):
    """Register Proxy

    This proxy catches lookups against non existent get_fieldname and set_fieldname methods
    and converts them into calls against the device's get_field and set_field methods with
    the appropriate options.

    This means device.register.set_field(value) and device.register.get_field(value) will work
    and also transparently update the underlying device without the register or field objects
    having to know anything about how data is written/read/stored.

    """

[Thynix]

I'm on board with that! Do you want to remove it, or I can open a PR?

[Gadgetoid]

Feel free to open a PR- though I’m going to have to do some careful inspection of our i2c-device based libraries to make sure I’m not wrong 😬

[PaulskPt]

Thank you gentlemen for giving your precious time to this!
Btw the script with the Pimoroni RTC breakout works perfect. Even having it left without electricity for weeks, it keeps time quit exactly.