MeshCore

Commit Graph

Author	SHA1	Message	Date
Mark Jocas	63fe3d42a2	Corrects typos and grammatical errors Improves readability across documentation and inline comments by fixing common spelling mistakes and duplicate words.	1 month ago
Josiah VanderZee	4b6b8abe7a	Add missing Wire.h include for sensors This slipped through in PR #2327 and I noticed because the TechoBoard.h for my variant doesn't include the Wire header, so the source file in question does not coincidentally obtain a copy.	2 months ago
Wessel Nieboer	3bde089bdb	Gate PIN_GPS_EN Do not reference it unconditionally	2 months ago
Kenneth Privitt	235706a22a	Incorrect time_t TypeDef in RAK12035_SoilMoisture	2 months ago
Nick Dunklee	c7be216f27	fix(sensors): improve sensor initialization and handling to prevent hangs and handle growth This is a medium-ish refactor to attempt to clean up sensor handling logic both for board stability and future potential growth before the code becomes all spaghetti and meatballs. I'd be curious to see if anyone running sensors out there that knows how to build and flash MeshCore code could give this a try and see how it behaves. It is working fine on my end on multiple nodes. PR notes are gigantic because it is a fundamental behavior repair for sensors, so I wanted to over-explain. Also, if it hadn't been mentioned previously, push-back is always welcome. I'm just spending my time trying to clean up / fix / enhance this corner of the firmware, and want to contribute my improvements back to the project. Problem: Current MeshCore code makes no attempt to see what sensors are actually available on the I2C bus at startup and blindly tries to interact with sensors. This has some very bad side-effects, like if a sensor that is unsupported, or has a weird initialization process, the MeshCore node will just hang at boot and never successfully start up and ostensibly looks bricked, or the INA226 and SHT4X both sharing the same address and the code just silently fighting. The current implementation also gloms sensor readouts from the MCU and environment sensors onto the same telemetry channel, with some arbitrary exceptions for incrementing channels based on certain behavioral situations. The MCU temperature and external temperature sensors would appear on channel 1, and it wouldn't be possible to tell which sensor the temperature value was coming from. Per [CayenneLPP](https://github.com/myDevicesIoT/CayenneLPP): Data Channel: Uniquely identifies each sensor in the device across frames, eg. “indoor sensor” So this channel division implementation falls inline with what CayenneLPP intended. There are up to 256 channels available. So I tried to model this change in that behavioral style. Proposed Improvement: This implementation scans the I2C bus for what devices are present, sets each sensor to its own CayenneLPP channel, and keeps MCU telemetry on channel 1 only. So Channel 1 is always "self" and no confusion can result. Details: - Channel 1 is always the MCU and things about it, so you always know that telemetry is from the board itself. Exception is GPS, GPS stays on channel 1 as well since it is "about the board" even though it's a bit gray-area as GPS can often be a secondary chip. - Each sensor board is allocated to a dedicated CayenneLPP channel, so if you are reading from that channel, you know the data is from that sensor only. (Sensors emitting more than one of the same type of measurement are exceptions.) - `scanI2CBus()` probes addresses 0x08–0x77 with raw `beginTransmission`/`endTransmission`. No sensor library is touched until after this completes. This will prevent sensor-based boot hangs, unknown or unresponsive devices never reach a library init call. - Created `SENSOR_TABLE` a compile-time array that is gated by the existing `ENV_INCLUDE_` macros. A sentinel `{ 0, nullptr, nullptr, nullptr }` at the end keeps the array non-empty regardless of which sensors are enabled, avoiding zero-length array warnings. - When `begin()` is called, scan first, then loop: skip if address not detected, skip if `init()` returns 0, otherwise register one ActiveSensor entry per sub-channel. - `querySensors()` I replaced the entire #ifdef chain with a 3-line loop. - T1000-E has its own T1000SensorManager, so it should be completely unaffected by this change. - SHT4X quirky initialization behavior is retained. - MLX90614 - git commits around this didn't have any notes as to why it is reporting ambient temperature on a separate channel as well as the object temperature, as the ambient temperature is used internally to compute the object temperature and not really needed for the sensor's purpose - just the same, kept the existing behavior of reporting the ambient temperature one channel above the channel assigned to the sensor - All `bool _initialized` fields are gone, replaced with `ActiveSensor _active_sensors[16]` (query function pointer and sub-channel index) and `_active_sensor_count. SensorDef` lives entirely in the `.cpp` so the header has no dependency on it. - Details on the INA226 and SHT4X: both default to address 0x44, the old code had a bug and would have both begin() calls fire and they would just fight each other silently. In the new code, the respective sensor code is only called if the device is actually present, however, if both were present simultaneously, SHT4X comes first in the table and would win, and INA226 would return false and be skipped. The INA226 has 16 possible addresses that are configurable in the hardware itself, so in a potential scenario where both sensors would be present, the person implementing that design could take that into account. - BME680 gas resistance will now transmit on the same channel as the rest of BME680 telemetry which is inline with CayenneLPP standards. Coupling this PR with https://github.com/meshcore-dev/MeshCore/pull/2146 streamline the whole sensor telemetry, and with https://github.com/meshcore-dev/MeshCore/pull/2149 will overall improve BME680 handling. The gas resistance sensor actually has a binary library to make it more useful, calibration, accounting for age of sensor, and other improvements, but since that adds more flash consumption, I have omitted that in PRs thus far. - RAK12035 and other current upstream dev branch changes integrated.	2 months ago
KPrivitt	27f732653b	Added RAK12035 Soil Moisture and Temperature Sensor (#2223 )	3 months ago
Quency-D	2cbe0c0398	Fixed the initialization error of the BME680 sensor.	3 months ago
Paul Moore	913a27da20	EnvironmentSensorManager: allow for INA3221 macro overrides Allow for platformio.ini files to override the default values for the following INA3221 related macros: TELEM_INA3221_ADDRESS TELEM_INA3221_SHUNT_VALUE TELEM_INA3221_NUM_CHANNELS Signed-off-by: Paul Moore <[email protected]>	4 months ago
Quency-D	cdca79540f	Update Heltec Tracker v2 to version KCT8103L.	4 months ago
Wessel Nieboer	10f2fecd45	Sync time with GPS every 30 minutes Unless your GPS is being spoofed there isn't really a downside to syncing more often with GPS. I understand the RTC is very stable, but especially with powersaving now clock drift is worse, we should sync more often.	6 months ago
Wessel Nieboer	8a9a0dca5f	Fix GPS +8mA power leak when disabled (nRF52) On the T114, GPS_RESET (pin 38) is the same pin as PIN_3V3_EN. MicroNMEALocationProvider::begin() sets pin 38 HIGH (powering the 3V3 rail) but stop() never set it back LOW, leaving the GPS module powered even when disabled. Assert reset pin in stop() to mirror begin(), and guard _location->loop() behind gps_active check. Fixes meshcore-dev/MeshCore#1628	5 months ago
Piotr Małek	3845a1c021	Fix incorrect INA260 address in debug message	5 months ago
Quency-D	fc61018d4d	Fix the issue of inconsistent I2C usage in the environmental sensor.	5 months ago
Frieder Schrempf	ab7935142c	EnvironmentSensorManager.cpp: Cleanup after failed RAK4631 GPS detection If no GPS was detected, revert the hardware to the initial state, otherwise we may see conflicts or increased power consumption on some boards. Signed-off-by: Frieder Schrempf <[email protected]>	7 months ago
Frieder Schrempf	e79ee11872	EnvironmentSensorManager.cpp: Fix RAK4631 serial GPS detection Serial1 is always true. If we want to check for the presence of a GPS receiver, we need to check if any data was received. Signed-off-by: Frieder Schrempf <[email protected]>	7 months ago
entr0p1	cc28b1a34d	EnvironmentSensorManager.cpp: Mitigate BME280 self-heating causing inaccurate readings.	6 months ago
agessaman	b91b854a1d	fix output from LPS22HB: convert barometric pressure from kPa to hPa in EnvironmentSensorManager	7 months ago
Scott Powell	d7adcc136b	* LPPDataHelpers, readCurrent() signed value	7 months ago
Christophe Vanlancker	01eb8716af	fix(core): optimize GPS loop and add display GPIO safeguards	7 months ago
csrutil	df3cb3d192	_location->loop() should be in the next tick	7 months ago
csrutil	62e180dc0f	changed ms to sec	7 months ago
csrutil	88fb173297	add configurable GPS update interval Make GPS update interval configurable via settings instead of using hardcoded 1 second value. The interval is persisted from preferences and can be adjusted at runtime through the sensor manager settings interface	7 months ago
Florent	1c0017b634	thinknode_m5: gps support	7 months ago
recrof	88a6141943	fix: move bme680 detection before bme280	7 months ago
kallanreed	9e61b56e70	Use correct macro for logging test	8 months ago
Wesley Ellis	4cfbd3bad5	Switch BMP085 mode to 0 for ULTRALOWPOWER	8 months ago
Wesley Ellis	ac15131296	Add support for bmp085/bmp180 temperature/pressure sensor	8 months ago
liquidraver	3c48f01601	BME680 library doesn't have altitude calculation, we can add it here to match other sensors'	9 months ago
liquidraver	0e7486552d	Add simple BME680 support to RAK with adafruit library	9 months ago
Scott Powell	d3be6afccb	* fix for non-RAK targets	9 months ago
Scott Powell	fa8c31be88	* fix for RAK12500 GPS (I2C)	9 months ago
Quency-D	70ac820594	add heltec tracker v2 board.	9 months ago
Florent de Lamotte	341b69e3c9	sensor list command	9 months ago
Florent	e4f2d63b0a	cli_gps: use sensormanger to toggle gps on/off to keep state coherent	9 months ago
Florent	0502bc370d	CommonCLI: gps management commands	9 months ago
fdlamotte	c83abbeff6	ESM: add gps reset after begin	9 months ago
fdlamotte	030f0d5d82	location provider: reduce reset delay	9 months ago
Florent	76aa7cf488	ui_task: initial gps page	9 months ago
Florent	c1915a1133	ESM: delegate gps management to LocationProvider	9 months ago
Florent	611d61b6c6	tracker_l1: fix bme226 init in ESM to include all sensors	9 months ago
Quency-D	669bea04a0	add heltec_v4 board.	9 months ago
Florent	f9543bb7bb	tracker_l1: support for EnvironmentSensorManager	9 months ago
Florent de Lamotte	d83cdc501f	ui: use LPPDataHelper and conditionals for sensors page	10 months ago
csrutil	cf93109cd5	✨ feat: add altitude support to environment sensor node telemetry - Include actual node altitude in GPS telemetry instead of hardcoded 0.0f - Extract altitude data from both ublox_GNSS and serial GPS sources - Update debug logging to display altitude alongside lat/lon coordinates	10 months ago
recrof	d59724acd0	new variant: RAK WisMesh Tag	10 months ago
Alex Wolden	0c37eafd01	Fixed shunt value for ina226	10 months ago
Alex Wolden	c636536599	Add INA226 to rak	10 months ago
Martin Kudlacek	822850b4d5	Support for Sensirion SHT4x	11 months ago
Martin Kudlacek	a96d1022a0	Added support for TI INA260	11 months ago
taco	ef58ef460b	fix: BMP280 altitude was using incorrect library	11 months ago

1 2

91 Commits (t-impulse-ui)