Fix #13, #14, and other improvements - Githubissues

Terrapin-Rocket-Team / Ground-Station

A Node.js/Electron ground station user interface to display and log APRS messages recieved over Serial

0 stars 0 forks source link

Fix #13, #14, and other improvements #15

Closed erodarob closed 5 months ago

erodarob commented 5 months ago

This pull request fixes #13 and fixes #14, alongside adding multiple additional improvements to the application frontend and backend.

Major

Resolved Regex matching issues during transmission reception, old transmissions should no longer be repeated
Removed any implicit requirements for messages to be 2 seconds apart, tested that transmissions can be received at up to 10Hz, but could go higher
Reworked chart implementation
Updated chart.js
Added apogee detection to the main screen

Minor

Received t0 is now interpreted as a time in UTC rather than local time
Added event sent to frontend signaling the window is about to close
Updated some dependencies
APRSMessage objects are now sent in to the frontend in debug mode, rather than plain Objects
Changed apogee detection to wait for 6 seconds before detecting apogee, rather than 2 transmissions
Added chart and apogee data to session storage so it is restored when the page is reloaded

Bug Fixes

Removed reference to windows after closing process was complete, rather than before

varun-un commented 5 months ago

Add altitude requirement for apogee