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

Sha256: Integrate horizon deactivation mechanism. #84

Merged
merged 12 commits into from
Mar 5, 2024
Merged

Sha256: Integrate horizon deactivation mechanism. #84

merged 12 commits into from
Mar 5, 2024

Conversation

realtimepeople
Copy link
Collaborator

@realtimepeople realtimepeople commented Feb 21, 2024

Checklist before merging

  • Is the most recent master version merged? Double check with git pull and git merge master.
  • Does it compile without errors?
  • Has someone reviewed your changes?

Hardware test

  • Flash the ESP and STM Firmware via USB and connect a GNSS antenna
  • Test without a SD-Card
  • Check that the Bluetooth and CAN works and transmitts data. If possible test with Hardware-version 1.0 and 2.0!
  • Insert a SD-Card with the latest sensor_config.txt file from Configuration_files and an logging enabled. Let the sensor run for ~ 30 minutes. Afterwards check that there are only two files: data file yymmdd_hhmmss.f* and a calibration/configuration dump file: yymmdd_hhmmss.EEPROM There shall be no crash-dump files.

@MaxBaex
Copy link
Member

MaxBaex commented Mar 5, 2024

Tested OK:

  • Working with and without SD Card.
  • NMEA output via Bluetooth with old and new sensor Hardware
  • NMEA output event without GNSS Fix
  • Different datarates for NMEA Data

@MaxBaex
Copy link
Member

MaxBaex commented Mar 5, 2024

  • 20 minutes logging and no crashdumps observed.

@MaxBaex
Copy link
Member

MaxBaex commented Mar 5, 2024

Script for binary creation does not work. Reported here: #75

@MaxBaex MaxBaex merged commit bbbb941 into master Mar 5, 2024
@realtimepeople realtimepeople deleted the sha256 branch March 28, 2024 10:24
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.

2 participants