rwaldron/johnny-five

BMP180 Temperature (and consequently Pressure) Error ( > about 40 °C )

Open

#1.696 geöffnet am 7. Aug. 2020

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (13.152 Stars) (1.845 Forks)batch import
BugClass:BarometerGood First IssueHardware Required

Beschreibung

The uncompensated temperature of the BMP180 is incorrectly converted to a 16-bit signed value, instead of a 32-bit signed value (long). Since the data is 16-bits wide and can therefore not be negative, the easy fix would be to do the following:

sip.js(1128): uncompensated = uint16(data[0], data[1]);

Please see page 15 of the manual of the BMP180 chip: https://ae-bst.resource.bosch.com/media/_tech/media/datasheets/BST-BMP180-DS000.pdf

Contributor Guide

Tech Stack
javascriptnodejs
Domain
backend
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
JavaScript
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
90
Research-Richtung
The issue identifies a bug in the BMP180 driver where uncompensated temperature is incorrectly converted to a 16 bit signed value instead of a 32 bit signed value. The suggested fix is to modify line 1128 of sip.js to use uint16() to read the raw data, then cast to a signed 32 bit integer. Review the BMP180 datasheet section on temperature measurement to confirm the correct data type. After making the change, test with temperatures above 40°C to ensure pressure readings are also corrected.