larus-breeze / sw_sensor

Firmware for Larus sensor MK1 and MK2
3 stars 3 forks source link

Individual configuration: Sensor may hang if no configuration given. #36

Closed realtimepeople closed 1 year ago

realtimepeople commented 1 year ago

Presently a virgin sensor MUST have read a meaningful set of configuration parameters as defined in the standard file sensor_config.txt.

We need to program something that make the sensor not to crash if no such file has been seen.

MaxBaex commented 1 year ago

I would say the sensor should, in that case use the very basic default configuration. Default mounting orientation, default time-constants, ... Ideally the sensor writes a default configuration file to the sd-card which the end can edit. It would help if the user does not need to search for an example file with the correct format.

MaxBaex commented 1 year ago

The sensor_config.txt file can be used to overwrite the compass calibration parameter which are optimized automatically during flight. There is the risk that users leave the sensor_config.txt file on the sd-card. This prevents that the calibration process works as the default magnetic scale and offset parameters are reset on every boot.

I suggest to only provide a sensor_config.txt file which does not write these continuously improving parameters. But which are these?

MaxBaex commented 1 year ago

There is a red error led on STM32:PD6 this led could be used to indicate that something e.g. the initial sensor configuration is missing.

realtimepeople commented 1 year ago

Solved with commit 8c37157