Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ds18b20 sensor #42

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

supersciencechris
Copy link

Pull Request Template

Description

Adds support for DS18B20 one wire temperature sensors. The default setting assumes a single sensor and does not require the user to go and find the address.
The optional one_wire_ID parameter is for the available for muliptle sensors (ie: one outside, one in the greenhouse, etc) but does require a lookup of the address. The docs include the
location and format of the address. This was tested on a Raspberry Pi 3 for about a year in a small cheap greenhouse where is got down to single digit temperature and over 100 in the summer
with fine performance, minus the occasional restart when the Pi likely was overheating.

Fixes # (issue)

@mentions the author(s)

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration including needed components.

  • Ran single ds18b20 sensor for ~12 months in small greenhouse in both extream heat and cold without issue.

Test Configuration:

  • Python version: 3.9.2
  • Hardware: Raspberry Pi 3 Model B Rev 1.2
  • Toolchain: VSCode, VIM
  • OS: Raspbian 11

New Pull Request Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have read the contributing documentation.
  • I have commented my code, particularly in hard-to-understand areas
  • I have provided corresponding changes to the documentation or included new files
  • My documentation follows the format and style of the example documentation.
  • I have tested that my fix is effective or that my feature works
  • This change/feature does not already exist and is not in another pending PR
  • Any dependent changes have been merged and published in downstream modules
  • I have checked my code and corrected any misspellings

Documentation

Please provide thorough documentation of your changes and features in a markdown format. You can find a example of the documentation here. If you are adding a new extension you may attach this as a separate file. It is important that you document your features well so that someone with no information can familiarize themselves and even help collaborate!

Screenshots / Media

If relevant, provide screenshots or any media that would help showcase the proposed changes.

SpaceCaseChris added a commit to SpaceCaseChris/mudpi-core that referenced this pull request Dec 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant