-
Notifications
You must be signed in to change notification settings - Fork 15
Add initial functionality for I2C and SPI. #1
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
Conversation
caternuson
commented
Nov 1, 2018
- basic readings: pressure, temperature, altitude
- over sampling individually settable for pressure and temperature
- IIR filter coefficient
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One thing but good otherwise.
adafruit_bmp3xx.py
Outdated
raise RuntimeError('Failed to find BMP3XX! Chip ID 0x%x' % chip_id) | ||
self._read_coefficients() | ||
self.reset() | ||
self.sea_level_pressure = 1013.25 # hPa @ MSL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a """comment""" after this so sea_level_pressure is shown in RTD.
i tested this code, seems to work fine so after the minor update, we can merge! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tested with hardware
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, looks good! Thanks!