adafruit_bme680.py, class Adafruit_BME680, _perform_reading(): infinite loop

[adafruit]

adafruit_bme680.py, class Adafruit_BME680, _perform_reading(): infinite loop

[caternuson]

Can you provide a short, simple, complete example sketch that can be used to demonstrate the issue.

[mtetcs]

In principle, the Usage Example in the docs is sufficient to demonstrate the issue, given the presence of the hardware in question. We only sample every 10s for the application here.

Perhaps the gist of the issue is that hardware isn't always 100% reliable. Arguably the best place to handle temporarily unresponsive hardware is in its API, rather than handling it downstream where it would necessitate an additional layer of complexity to forcibly terminate hung read operations. _perform_reading() could plausibly raise a BlockingIOError if it doesn't receive an acceptable response after a given number of iterations, in preference to an infinite loop.

[caternuson]

Is the inclusion of the TCA9548A necessary? Or do you get the same issue even without the TCA being used?

[mtetcs]

No and no. There was no issue with the BME680 hardware in either configuration.

[caternuson]

OK, if there's no issue, then not sure what the issue is?

Is there any issue when using just a single BME680 (no TCA9548A) and running the example code from this library?
https://github.com/adafruit/Adafruit_CircuitPython_BME680/blob/main/examples/bme680_simpletest.py

[mtetcs]

The issue reported is that _perform_reading() fails to consider the possibility that the hardware may be non-responsive. Instead of failing gracefully and allowing the caller to deal with the problem, it enters an infinite loop, halting progression.