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

Area of Interest Management #12

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

Conversation

chgio
Copy link
Member

@chgio chgio commented Sep 10, 2023

As per #7, this MR introduces the implemented management of Areas of Interest, with telecommand-based configuration (configure, enable, disable) and automated comparison with the current location.

Giorgio Ciacchella added 5 commits March 30, 2023 15:18
overhaul AoI as 4 locations + enabled flag:
- the 4 locations represent the corners of a bounding box
- add decision logic to aoi_manager to compare against all 4 corners
- add decision logic to aoi_manager to compare against the minimum and
  maximum altitudes out of the 4 corners
- add enabled flag to AoI
- add logic to aoi_manager to only consider an AoI for decision if its
  enabled flag is true

overhaul AoI config TC as one of:
- configure (legacy add)
- enable: set enabled flag for AoI of specified index
- disable: reset enabled flag for AoI of specified index

switch location interfaces from "interrupting" to "polled":
- move cyclical location request trigger from location_tracker to
  aoi_manager
- add request_location interface, cyclically triggered
- trigger location_tracker's location sending with location request

in aoi_manager, also change loop keyword and remove excessively simple
procedures

also remove print_service and its references, and add SDL x86
implementation to HAL for simulation
roll back docker configs to using the feature_bullseye upstream for
everything (Michal's forks got merged)
@chgio chgio added the feature Implementation of a new feature from the Capella model label Sep 10, 2023
@chgio chgio self-assigned this Sep 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Implementation of a new feature from the Capella model
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant