GitLab

Closes #78 (closed)

changed milestone to %CB-2024W37

added Partner::NOC Project::MAS-DT labels

added 1 commit

• a1880a9e - feat: add waypoint status message for dry-run

Compare with previous version

@owanes - done what we discussed yesterday.

• waypoint_status shares the same format as platform_status so we only need one parser in c2 land
• waypoint_status requires either waypoint or waypoint_latitude + waypoint_longitude
• platform_status requires either position or latitude + longitude or no_position_reason which is an enum.

I'm still a little concerned about the one parser idea because we want to include the current waypoint in the platform_status for ORI but I don't think we want to ingest the waypoint from every platform_status into positions

added 1 commit

• 4c46abdb - refactor: move waypoint_status schema

Compare with previous version

added 1 commit

• 50ada7fd - refactor: new waypoints message

Compare with previous version

added 1 commit

• 85aec3d0 - chore: lint with black

Compare with previous version

marked this merge request as ready

After more talk we've gone back to preferring simulated platform_status for this since during the surface dialog after transferring and re-reading the ma files there should be another real platform_status containing the new waypoint.

We can use the changes in the platform_status message in this MR for that purpose but we decided to leave the waypoints message in place for now but with no immediate plans to use it.

requested review from @trishna

added 20 commits

• 85aec3d0...bab76ac7 - 14 commits from branch 67-mas-dt
• a0ae5a20 - feat: add waypoint status message for dry-run
• 58f07b32 - refactor: move waypoint_status schema
• 4a5d19a4 - refactor: new waypoints message
• f419d7c3 - chore: lint with black
• c5c9e77e - feat: add enum options for no position
• 1d51a7b2 - fix: correct uuid for example messages

Compare with previous version

added 1 commit

• 6901b7d7 - chore: run black

Compare with previous version

resolved all threads

approved this merge request

mentioned in commit c37ec78c

merged