fix: fixed BMP180

[AsCress]

Fixes the BMP180 sensor.
The following libraries were used as a reference: -

1. https://github.com/adafruit/Adafruit-BMP085-Library
2. https://github.com/adafruit/Adafruit_Python_BMP

Screenshots / Recordings

Here, is a screenshot of the readings. These are consistent with the results provided by other python and Arduino drivers.

@bessman Please let me know your views on this.

Summary by Sourcery

Fix BMP180 sensor functionality and refactor the BMP180 class to enhance readability and support multiple operating modes.

Bug Fixes:

• Fix BMP180 sensor readings to ensure accurate temperature and pressure measurements.

Enhancements:

• Refactor BMP180 class to improve readability and maintainability.
• Add support for different operating modes (ultra-low power, standard, high resolution, ultra-high resolution).
• Improve calibration data handling and calculations for temperature and pressure.

[sourcery-ai]

Reviewer's Guide by Sourcery

This pull request refactors the BMP180 sensor class to improve its accuracy and readability. The changes include the addition of module-level constants, new methods for reading calibration values, updated temperature and pressure calculations, and the removal of redundant code. These updates are based on the BMP180 datasheet and reference libraries from Adafruit.

File-Level Changes

Files	Changes
pslab/external/BMP180.py	Refactored the BMP180 sensor class to improve accuracy and readability by using constants, new calibration methods, and updated calculations based on the datasheet.

---

Tips

• Trigger a new Sourcery review by commenting @sourcery-ai review on the pull request.
• Continue your discussion with Sourcery by replying directly to review comments.
• You can change your review settings at any time by accessing your dashboard:
  • Enable or disable the Sourcery-generated pull request summary or reviewer's guide;
  • Change the review language;
• You can always contact us if you have any questions or feedback.

[sourcery-ai]

Hey @AsCress - I've reviewed your changes - here's some feedback:

Overall Comments:

• Consider replacing the print statement in init with proper logging for debugging information.
• Great job on improving the code structure and accuracy. Consider adding more type hints to further enhance code readability and maintainability.

Here's what I looked at during the review

• 🟡 General issues: 1 issue found
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.}

[bessman]

Apologies for the late review, I've been traveling this week with spotty internet access.

[AsCress]

@bessman I have (tried to) implement all the changes you suggested. This PR is now up-to-date with my changes.

[AsCress]

@bessman I've made all the changes that we discussed and that you've suggested. Now, the pressure, temperature and altitude can be accessed as properties: -

Let me know if there's something still not quite right style-wise :))

[bessman]

Looks good, thank you!