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

[GS] Add data screen with flight statistics #386

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

l-jost
Copy link
Member

@l-jost l-jost commented Sep 22, 2024

Description

This MR adds the data screen on the ground station with flight statistics

Checklist for changes

  • Feature tested manually [Mandatory]
  • [na] Simulation flight performed [Optional]
  • [na] Drone flight performed [Optional]
  • Copyright notice for new files
  • #pragma once directive for new header files

Related issues

Closes #305

@l-jost l-jost force-pushed the gs-add-data-screen branch 3 times, most recently from 9c67449 to 6bc34ea Compare September 22, 2024 07:03
@l-jost l-jost force-pushed the gs-add-data-screen branch 4 times, most recently from 8d8ea8c to ad6a3b6 Compare September 22, 2024 10:59
@l-jost l-jost self-assigned this Sep 22, 2024
@l-jost l-jost changed the title [Draft] Add data screen for ground station [GS] Add data screen with flight statistics Sep 22, 2024
Comment on lines +324 to +326
// The display can show a maximum of 11 entries, no scrolling implemented!
// It is highly unlikely to get more than 11 flight logs because the flash is quite small
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if we do have more than 11 files, do we display only the newest or oldest files?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

They are sorted alphabetically, with how the flights are currently names, it would show the 11 oldest flights

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.

[GS] Implement Data Panel
2 participants