Should the implementation of MCP230xxBase#input_pins() be consistent with the local buffers (self.iodir, self.gppu) where the local copy of the self.gpio bytes should be the last known state?

Currently, in input_pins() when gpio is read, the resulting bytes are not stored to self.gpio but instead to a gpio variable with local scope, meaning the read may not have the side-effect of updating the MCP230xxBase object's knowledge of the current GPIO state as some may expect.