Create initial message schemas

Closes #7 (closed)

examples/ecosub_c2/platform_status-from_usbl_example.json

+{
+    "message_ID": "b427003c-0000-11aa-a1eb-bvcdfghjgfdd",
+    "timestamp": "2022-11-16T00:00:00Z",
+    "version": 2,
+    "source": "hermes",
+    "destination": "autonomy_engine",
+    "delivery_type": "publish",
+    "encoded": false,
+    "message_type": "platform_status",
+    "platform_ID": 1,

examples/ecosub_c2/observation.json

+{
+    "message_ID": "b427003c-0000-11aa-a1eb-bvcdfghjgfdd",
+    "timestamp": "2022-11-16T00:00:00Z",
+    "version": 2,
+    "source": "ecosub_c2",
+    "destination": "autonomy_engine",
+    "delivery_type": "publish",
+    "encoded": false,
+    "message_type": "observation",
+    "platform_ID": 3,

examples/gui/planning_configuration.json

+              "safe_command": "go_home",
+              "target_depth": 10.0
+            },
+            "max_velocity": 0.9,
+            "min_altitude": 15.2,
+            "min_velocity": 0.1,
+            "additional_data": {
+                "scan_type": "DVL",
+                "swath_width": 10
+            }
+          },
+          {
+            "platform_ID": 10,
+            "serial": "ecosub-5",
+            "model": "ecosub",
+            "emergency": {

formats/mission_plan.py

+            "example": 15.0,
+        },
+        "activate_payload": {
+            "type": "boolean",
+            "description": "To activate/deactivate sensor for Autosub "
+            + "Hover-1 --> `MBES` sensor and for EcoSUB --> `Sidescan`",
+            "example": True,
+        },
+        "send_environmental_data": {
+            "type": "boolean",
+            "description": "To trigger the platform to send list of"
+            + "  observations if any found",
+            "example": False,
+        },
+    },
+    "required": [

formats/mission_plan.py

+action_schema = {
+    "type": "object",
+    "properties": {
+        "action": {
+            "type": "string",
+            "description": "Autonomy Engine's action from `move`, `payload`,"
+            + " `dive`, `send_hits`, `scanline`, `scanpoint`.",
+            "example": "move",
+        },
+        "flight_style": {
+            "type": "string",
+            "description": "Platform-specific modes/flight styles to perform"
+            + " next action",
+            "example": "orbit",
+        },
+        "latitude_waypoint": {

formats/platform_status.py

+            "example": {"payload": [1.2, 434]},
+        },
+    },
+    "required": [],
+}
+
+platform_status_message_schema = {
+    "allOf": [{"$ref": "#/components/schemas/Message"}],
+    "type": "object",
+    "properties": {
+        "platform_ID": {
+            "type": "integer",
+            "description": "Identifier for platform",
+            "example": 1,
+        },
+        "status_source": {

formats/planning_configuration.py

+            "description": "Minimum altitude set for squad.",
+            "example": 15.2,
+        },
+        "min_velocity": {
+            "type": "number",
+            "format": "float",
+            "description": "Minimum velocity set for squad.",
+            "example": 0.1,
+        },
+        "max_velocity": {
+            "type": "number",
+            "format": "float",
+            "description": "Maximum altitude set for squad.",
+            "example": 0.9,
+        },
+        "additional_data": {

requirements.txt

+Flask
+flask-restx
+flasgger
+flask-marshmallow

tests/test.py

+"""
+schema, spec_url = read_from_filename("fixtures/swagger.json")
+errors_iterator = openapi_v3_spec_validator.iter_errors(schema)
+print(errors_iterator)
+print(openapi_v30_spec_validator.validate(schema))
+
+"""
+Test schema - Hydrosurv Messages
+"""
+message_types = ["acknowledgement", "mission_plan", "platform_status"]
+for item in message_types:
+    f = open("mock_data/hydrosurv_adapter/%s.json" % item)
+    mock_data = json.load(f)
+    header_data = mock_data["header"]
+    payload_data = mock_data["payload"]
+    print(validate(header_data, message_header, format_checker=FormatChecker()))

docs/generate_swagger.py

         "type": fields.String(
             required=True,
             description="Type of message",
-            example="vehicle_status",
+            example="platform_status",
         ),
         "payload": fields.Raw(required=True)  # TODO: schema applicable changes according to "type"
     },
         "type": fields.String(
             required=True,
             description="Type of message",
-            example="vehicle_status",
+            example="platform_status",
         ),
         "payload": fields.Raw(
             required=True,
     },
 )
 
-vehicle_schema = api.model(
-    "VehicleSchema",
+platform_schema = api.model(
+    "PlatformSchema",
     {
-        "vehicle_ID": fields.Integer(
+        "platform_ID": fields.Integer(
             required=True,
-            description="unique identifier for vehicle",
+            description="unique identifier for platform",
             example="ah-1",
         ),
         "serial": fields.Integer(
             required=True,
-            description="vehicle serial number",
+            description="platform serial number",
             example="ah-1",
         ),
         "model": fields.Integer(
             required=True,
-            description="vehicle serial number",
+            description="platform serial number",
             example="ah-1",
         ),
         "constraints": fields.Nested(constraints_schema),
         "active": fields.Boolean(
             required=False,
-            description="When a vehicle is in deployment (executing a mission plan) this should be True",
+            description="When a platform is in deployment (executing a mission plan) this should be True",
             example=True,
         ),
     },
             description="Identifier of given squad",
             example="ah-1",
         ),
-        "no_of_vehicles": fields.Integer(
+        "no_of_platforms": fields.Integer(
             required=True,
-            description="number of vehicles",
+            description="number of platforms",
             example="ah-1",
         ),
-        "vehicles": fields.List(
-            fields.Nested(vehicle_schema),
+        "platforms": fields.List(
+            fields.Nested(platform_schema),
             required=True,
-            description="Squad consists of these vehicles",
+            description="Squad consists of these platforms",
         ),
         "squad_mission_type": fields.String(
             required=True,
     {
         "gps_source": fields.Float(  # TODO: TBD with partners
             required=False,
-            description="Source of gps position. E.g. USBL (external), vehicle itself (internal)",
+            description="Source of gps position. E.g. USBL (external), platform itself (internal)",
             example="internal",
         ),
         "latitude_type": fields.String(
             required=True,
             description="Message header",
         ),
-        "vehicle": fields.Nested(vehicle_schema),
+        "platform": fields.Nested(platform_schema),
         "time": fields.String(
             required=True,
             description="Timestamp of message",
         # properties of each observation?
         "points_of_interest": fields.Float(
             required=False,
-            description="Points from features of interest identified by vehicle if any found. DEFINE FORMAT.",
+            description="Points from features of interest identified by platform if any found. DEFINE FORMAT.",
             example="",
         ),
         "region_surveyed": fields.Float(
             required=False,
-            description="Region surveyed by given vehicle. DEFINE FORMAT. GEOJSON?",
+            description="Region surveyed by given platform. DEFINE FORMAT. GEOJSON?",
             example="",
         ),
         "quality_of_points": fields.Float(
             required=False,
-            description="Quality/strength of points from features of interest identified by vehicle. DEFINE FORMAT.",
+            description="Quality/strength of points from features of interest identified by platform. DEFINE FORMAT.",
             example=0.98,
         ),
         "additional_data": fields.Raw(
         ),
     }
 )
-vehicle_status_message_schema = api.model(
-    "VehicleStatusMessage",
+platform_status_message_schema = api.model(
+    "platformStatusMessage",
     {
         "message": fields.Nested(
             message_header_schema,
             required=True,
             description="Message header",
         ),
-        "vehicle": fields.Nested(vehicle_schema),
-        "vehicle_state": fields.String(
-            # TODO: Define dictionary with potential STATES of each vehicle
+        "platform": fields.Nested(platform_schema),
+        "platform_state": fields.String(
+            # TODO: Define dictionary with potential STATES of each platform
             required=False,
-            description="Current state executed by vehicle. E.g. "
+            description="Current state executed by platform. E.g. "
             + "STOP, IDLE, ABORT.",
             example="IDLE",
         ),
         "autonomy_mission_ID": fields.Integer(
             required=False,
-            description="Last mission plan ID (according to Autonomy Engine's mission plan number) executed by vehicle",
+            description="Last mission plan ID (according to Autonomy Engine's mission plan number) executed by platform",
             example=1,
         ),
-        "mission_track_number": fields.Integer(
+        "mission_track_ID": fields.Integer(
             required=False,
             description=(
                 "Track number - stage in mission (e.g. "
             ),
             example=4,
         ),
+        "mission_action_ID": fields.Integer(
+            required=False,
+            description="to add description",
+            example=1,
+        ),
         "range_to_go": fields.Float(
             required=False,
             description="Estimated distance to reach next waypoint",
         ),
         "health_status": fields.String(
             required=False,
-            description="Health status extracted by respective vehicle "
+            description="Health status extracted by respective platform "
             + "if any diagnosis available checks on sensors",
             example="Warning",
         ),
         "gps_data": fields.List(
-            fields.Nested(gps_schema),  # TODO: TBD Do we want a list of gps readings to allow > 1 reading i.e. vehicle + usbl
+            fields.Nested(gps_schema),  # TODO: TBD Do we want a list of gps readings to allow > 1 reading i.e. platform + usbl
             required=True,
-            description="Metadata pf each vehicle",
+            description="GPS of each platform",
         ),
         "localisation_error": fields.Float(
             required=False,
             example="",
         ),
         "battery_remaining_capacity": fields.Float(
-            required=False,
-            description="Battery remaining capacity % provided by respective C2 if any.",
+            required=True,
+            description="Battery remaining capacity % provided by respective platform/C2.",
             example=80.0,
         ),
         "sensor_config": fields.Nested(sensor_schema),  # TODO: TBD Do we want a list of sensors to allow > 1 sensor
             description="",
             example="",
         ),
-        "vehicle_ID": fields.Integer(
+        "platform_ID": fields.Integer(
             required=False,
             description="Details of each squad",
         ),
         pass
 
 
-ns2 = api.namespace("vehicle_status", description="vehicle Status Message Format")
+ns2 = api.namespace("platform_status", description="platform Status Message Format")
 @ns2.route("")
-class VehicleStatus(Resource):
-    @ns2.response(200, "Success", vehicle_status_message_schema)
+class platformStatus(Resource):
+    @ns2.response(200, "Success", platform_status_message_schema)
     def get(self):
         pass
 
 
 ns3 = api.namespace(
-    "autonomy_configuration", description="Autonomy Configuration Format. Do we want region of interest to be per squad, per vehicle or in the main schema?"
+    "autonomy_configuration", description="Autonomy Configuration Format. Do we want region of interest to be per squad, per platform or in the main schema?"
 )
 
 @ns3.route("")
         pass
 
 
-# @api.route('/mission-plan/<str:vehicle_type')
-# @api.doc(params={"vehicle_type": "The type of vehicle of the mission plan to target."})
-ns4 = api.namespace("mission_plan", description="Mission Plan Format Per vehicle")
+# @api.route('/mission-plan/<str:platform_type')
+# @api.doc(params={"platform_type": "The type of platform of the mission plan to target."})
+ns4 = api.namespace("mission_plan", description="Mission Plan Format Per platform")
 
 @ns4.route("")
 class MissionPlan(Resource):
     def get(self):
         pass
 
-ns5 = api.namespace("observation", description="Observation Format --> Per vehicle or generic?")
+ns5 = api.namespace("observation", description="Observation Format --> Per platform or generic?")
 @ns5.route("")
 class Observation(Resource):
     @ns5.response(200, "Success", observation_schema)

examples/autonomy_engine/configuration.jsonc

-//****************************************************************
-// File: config.json  (THIS FILE WILL BE IN JSON FORMAT, NOT JSONC)
-// Description: This file is configured by the respective
-// vehicle's C2 if a vehicle is no longer active or change
-// a configured value. 
-//
-// Notes: All comments in this document will be stripped out
-// during SOAR operations.
-// (TODO: RHU to identify values that can and cannot be
-// configured).
-//----------------------------------------------------------------
-// CHANGELOG
-// -------------------
-// 11-11-2022 Trishna S.  Added comments, header, and data types  
-//                        for each field.
-// 10-11-2022 Izzat K.    Converted contents to json schema.
-// 06-10-2022 Trishna S.  New initialisation file.
-//****************************************************************
-{
-    "mission_config": {
-        "execute_stage": {
-            "survey": true, // bool
-            "detail": false // bool
-        }
-    },
-    "survey_team": {
-        "no_of_vehicles": 3, // int
-        "vehicles": [
-            {
-                "id": 1, // int
-                "vehicle": "ecosub", // str
-                "vehicle_serial": "eco101", // str
-                "active": true // bool
-            },
-            {
-                "id": 2,
-                "vehicle": "ecosub", // str
-                "vehicle_serial": "eco-102",  // str
-                "active": true // bool
-            },
-            {
-                "id": 3, // int
-                "vehicle": "ecosub", // str
-                "vehicle_serial": "ecosub3", // str
-                "active": true // bool
-            }
-        ],
-        "min_altitude": 0.0, // float
-        "min_velocity": 0.0, // float
-        "max_velocity": 0.0 // float
-    },
-    "detail_team": {
-        "execute_stage": true, // bool
-        "no_of_vehicles": 1, // int
-        "vehicles": [
-            {
-                "id": 1, // int
-                "vehicle": "autosub-hover", // str
-                "vehicle_serial": "ah1", // str
-                "active": true // bool
-            }
-        ],
-        "min_altitude": 0.0, // float
-        "min_velocity": 0.0, // float
-        "max_velocity": 0.0 // float
-    },
-    "environment": {
-        "region_of_interest": 0.0, // float
-        "exclusion_zone": 0.0 // float
-    }
-}
\ No newline at end of file

examples/autosub_c2/sample_decoded_ah1_status.json

-{
-    "reason": "ready",
-    "gps_source": "internal",
-    "gps_lat": 60.16048651129611,
-    "gps_lon": -1.156730960091541,
-    "gps_fix_seconds_ago": 1665580800,
-    "alr_data": {
-      "format_version": "ocsv3",
-      "message_type": "H",
-      "message_source": 3,
-      "transmit_timestamp": 1665580753.564244,
-      "content_mask": 18410715202459939000,
-      "depth": -1.304158329963684,
-      "water_conductivity": 0,
-      "water_temperature": 0,
-      "max_depth": 0,
-      "altitude": 1000,
-      "min_altitude": 0,
-      "vehicle_heading": -58.08430862426758,
-      "water_speed": -0.000008320044798892923,
-      "time_since_last_gps": 1665580800,
-      "time_on_surface": 7814.99365234375,
-      "w_hours": 28.418603897094727,
-      "battery_current": 0.4806833267211914,
-      "battery_max_current": 65.45500183105469,
-      "battery_voltage": 25.113433837890625,
-      "battery_min_voltage": 23.034000396728516,
-      "leak_sensor_output": 9,
-      "ground_fault_sensor_out": 752,
-      "propeller_rpm": 0,
-      "rs485_fault_flags": {},
-      "rs232_fault_flags": {},
-      "helm_state": {
-        "8": "idle"
-      },
-      "max_abort_link_state": 358,
-      "health_events": {},
-      "helm_events": {
-        "14": "waiting"
-      },
-      "log_storage_space": 98649407488,
-      "sbd_parser_error": {},
-      "alr_warning": 0,
-      "compass_align_value": -1.8788431882858276,
-      "battery_energy_used": 0,
-      "dead_nodes": {},
-      "position": {
-        "latitude": 60.16048651129611,
-        "longitude": -1.156730960091541
-      },
-      "pth_1": {
-        "pressure_temp": 2.0399177074432373,
-        "pressure": 593,
-        "temp": 17.80158233642578,
-        "rel_humidity": 7
-      },
-      "pth_2": {
-        "pressure_temp": 1.9932914972305298,
-        "pressure": 574,
-        "temp": 14.916650772094727,
-        "rel_humidity": 16
-      },
-      "rs232_mute_counts": {
-        "port00": 0,
-        "port01": 0,
-        "port02": 0,
-        "port03": 0,
-        "port04": 0,
-        "port05": 0,
-        "port06": 0,
-        "port07": 0,
-        "port08": 0,
-        "port09": 0,
-        "port10": 0,
-        "port11": 0,
-        "port12": 0,
-        "port13": 0,
-        "port14": 0,
-        "port15": 0
-      },
-      "rs485_mute_counts": {
-        "port00": 0,
-        "port01": 0,
-        "port02": 0,
-        "port03": 0,
-        "port04": 0,
-        "port05": 0,
-        "port06": 0,
-        "port07": 0,
-        "port08": 0,
-        "port09": 0,
-        "port10": 0,
-        "port11": 0,
-        "port12": 0,
-        "port13": 0,
-        "port14": 0,
-        "port15": 0
-      },
-      "helm_state_bitmask": 8388608,
-      "helm_state_bitstring": "00000000100000000000000000000000",
-      "helm_events_bitmask": 562949953421312,
-      "helm_events_bitstring": "0000000000000010000000000000000000000000000000000000000000000000",
-      "health_events_bitmask": 0,
-      "health_events_bitstring": "0000000000000000000000000000000000000000000000000000000000000000",
-      "sbd_parser_error_bitmask": 128,
-      "sbd_parser_error_bitstring": "10000000",
-      "dead_nodes_bitmask": 0,
-      "dead_nodes_bitstring": "0000000000000000000000000000000000000000000000000000000000000000",
-      "rs232_fault_flags_bitmask": 0,
-      "rs232_fault_flags_bitstring": "0000000000000000",
-      "rs485_fault_flags_bitmask": 0,
-      "rs485_fault_flags_bitstring": "0000000000000000",
-      "rudder_status": {
-        "errors": [
-          "current_info_not_received"
-        ],
-        "error_bitmask": 16,
-        "error_bitstring": "00010000",
-        "angle": 0
-      },
-      "sternplane_status": {
-        "errors": [
-          "current_info_not_received"
-        ],
-        "error_bitmask": 16,
-        "error_bitstring": "00010000",
-        "angle": 0
-      },
-      "auv_roll": -0.8640746474266052,
-      "auv_pitch": 0.2900053858757019,
-      "biocam_status": {
-        "biocam_current_mode": 9,
-        "biocam_cam_0_image_count": 0,
-        "biocam_cam_1_image_count": 0,
-        "biocam_cpu_temperature": 273,
-        "biocam_cam_0_temperature": 273,
-        "biocam_cam_1_temperature": 273,
-        "biocam_available_disk_space": 0
-      },
-      "serial_states": {
-        "port01": {
-          "3": "on"
-        },
-        "port02": {
-          "3": "on"
-        },
-        "port03": {
-          "0": "off"
-        },
-        "port04": {
-          "0": "off"
-        },
-        "port05": {
-          "0": "off"
-        },
-        "port06": {
-          "0": "off"
-        },
-        "port07": {
-          "0": "off"
-        },
-        "port08": {
-          "0": "off"
-        },
-        "port09": {
-          "0": "off"
-        },
-        "port10": {
-          "3": "on"
-        },
-        "port11": {
-          "3": "on"
-        },
-        "port12": {
-          "0": "off"
-        },
-        "port13": {
-          "3": "on"
-        },
-        "port14": {
-          "0": "off"
-        },
-        "port15": {
-          "0": "off"
-        },
-        "port16": {
-          "0": "off"
-        }
-      },
-      "computed": {}
-    },
-    "serial_number": "alr-3",
-    "time": 1665580753.564244,
-    "version": "ocsv3",
-    "_kind": "comms",
-    "autosub_message_format": {
-      "git_commit_hash": "ba3e33a9e786772d9642390711cf48b0a08755ea"
-    }
-}
\ No newline at end of file

formats/__init__.py

     if x.endswith(".py") and x != "__init__.py"
 ]
 
+# TODO: Define units for all schemas
+
 message_types = [
-    "vehicle_status",
+    "platform_status",
     "mission_plan_ecosub",
     "mission_plan_reav",
     "mission_plan_autosub",
         "type": fields.String(
             required=True,
             description="Type of message",
-            example="vehicle_status",
+            example="platform_status",
         ),
         "payload": fields.Raw(
             required=True,
             description="Content of Message",
             # example="{}",
         ),
-    }
+    },
 )
 
 constraints_schema = api.model(
     },
 )
 
-vehicle_schema = api.model(
-    "vehicleSchema",
+platform_schema = api.model(
+    "PlatformSchema",
     {
-        "vehicle_ID": fields.Integer(
+        "platform_ID": fields.Integer(
             required=True,
-            description="unique identifier for vehicle",
+            description="unique identifier for platform",
             example="ah-1",
         ),
         "serial": fields.Integer(
             required=True,
-            description="vehicle serial number",
+            description="platform serial number",
             example="ah-1",
         ),
         "model": fields.Integer(
             required=True,
-            description="vehicle serial number",
+            description="platform serial number",
             example="ah-1",
         ),
         "constraints": fields.Nested(constraints_schema),
         "active": fields.Boolean(
             required=False,
-            description="When a vehicle is in deployment (executing a mission plan) this should be True",
+            description="When a platform is in deployment (executing a mission plan) this should be True",
             example=True,
         ),
     },
 )
-# TODO: Add generic positions schema
-# TODO: Define units for all schemas

[Trishna Saeharaseelan]

Note: Ignore all the files under examples/*. Main files are:

• README.md
• init.py
• autonomy_configuration.py
• platform_status.py

[izzat]

Another thing to consider is the sensor data for perception or scientific observation, and whether it should go under platform status or a separate status message/topic for on-board sensor payload. Another thing we discussed previously is if this could be made generic or if we should allow customised data/messages for some platforms.

For SoAR, this would be for the pings/targets/feature-of-interests detected by the AUVs.

[Trishna Saeharaseelan]

[Alvaro] Add message format version

[Trishna Saeharaseelan]

To add separate message schema for usbl_position_updates

latitude: <>
longitude: <>
depth: <>
timestamp: <>
comms_quality: <>
ref_platform_ID: <>

[Trishna Saeharaseelan]

@geosal to have a look for USBL positions update example.

[Trishna Saeharaseelan]

@geosal to have a look at observations example. Is anything missing?

[Trishna Saeharaseelan]

@geosal @danjon to check out initial sample based on our idea for emergency behaviour in the planning config

[Trishna Saeharaseelan]

Remove safe_command

[Trishna Saeharaseelan]

[DAN] Useful to add a heading value in the vehicle status updates

[James Kirk]

Should action also be required? I guess the sub can declare its position without that, but I'd think that might be an action in and of itself? declare_pos action or something?

[Trishna Saeharaseelan]

[TO DISCUSS] action is basically from the Autonomy engine's side (i.e. what is the "action" in autonomy engine lingo). Let's discuss?

[Trishna Saeharaseelan]

To be platform-agnostic; remove flightstyle field + make action mandatory

declare_pos > not needed

[James Kirk]

Few comments of little bits I think have been from copy + pasting Only other thing is there's a lot of TODOs, commented out stuff, and just things to come back to later. Are the comments needed in there? And are there issues or notes somewhere of what work is left to do amongst all the TODOs and that?

[Dan Jones]

Can we have a chat about the structure of the messages maybe after stand-up tomorrow? This (I think) is the opposite way round from the schema I've based the prototype on. In the prototype schema every message is an instance of Message and the anyOf defines which payload schema it contains. The idea is that the backbone only deals in messages and passes the message on to the protocol handler which cares about what a message contains.

The model here is there are a bunch of different top-level schemas which each contain the content of the "Message" schema. It may well be fine or even preferable but it doesn't match the thing I've currently got working so it's probably worth a conversation.

[Trishna Saeharaseelan]

[TO DISCUSS] Interesting. Let's discuss this (need to clarify). If we need to change it from this end instead of the core, we should make the changes sooner since the partners will be working with this schemas as a start

[Trishna Saeharaseelan]

• Trishna to follow the structure the adapter is using to read schemas
• Validate all examples

[Dan Jones]

Have you tried validating the examples against this schema? I had a go with npm swagger-model-validator which I've used for the javascript adapter in the prototype. So far I've only tried the ecosub platform status message but it's not successfully validating.

[Trishna Saeharaseelan]

So with openapi-spec-validator it was validated successfully. Let's chat. That being said, I will add a task to build some tests

[Dan Jones]

That's interesting. I'll see if there's a javascript equivalent and have a go at validating it with that.

[Dan Jones]

Oh wait a minute. openapi-spec-validator is validating that the swagger.json conforms to the open api standard. It's not parsing a messaging and validating that the message conforms to the schema definition. What I'm talking about is validating that the example platform_status.json messages validate against the PlatformStatus definition defined in the schema.

[Dan Jones]

I think the python version of what I'm talking about is here: https://github.com/p1c2u/openapi-schema-validator

[Dan Jones]

The swagger.json is also curiously different in structure to all out existing APIs. Normally the models are under definitions but here they're under components/schemas and definitions is there but empty.

[Dan Jones]

The other thing Ashley suggested was that rather than define our own schema for anything geographic we just point to the existing GeoJson feature schema: https://geojson.org/schema/Feature.json

We could explicitly require a Point or a BoundingBox or whatever or just point to the parent feature definition.

Point is not very specific - it just has a min of 2 coordinates so we could supply 3=z or even 4=depth+altitude if it were necessary.

[Dan Jones]

The advantage of using Feature would be that the operating area and exclusion zones could be a BoundingBox or they could be a Polygon or even a MultiPolygon. We might not want to allow that complexity for the MVP but it would be good to have it in future - especially given the ultimate goal of the funders.

[Trishna Saeharaseelan]

Actually geojson is being in the planning_configuration message for exclusion_zones and the optional region_of_interest. Would we want to generate a 4-point polygon for each point then?

[Dan Jones]

What I mean by using geojson is adding a ref to the published geojson schema.

"coordinates": {
  "$ref": "https://geojson.org/schema/Point.json"
},

What the planning configuration message is doing is creating a data structure that matches GeoJSON.

To do that we'd need a validator that supported resolving remote refs which it seems like the one I'm currently using doesn't.

[Dan Jones]

Would we want to generate a 4-point polygon for each point then

No - We use the GeoJSON feature that represents what we want to draw. So Point for a status update/area of interest, Polygon for an operating area / exclusion zone.

[Dan Jones]

This one does it: https://www.npmjs.com/package/json-schema-remote

[Trishna Saeharaseelan]

@allore what's the best terminology to distinguish between the type of status message (i.e. from USBL or from the platform itself) vs. source/rx mode of the message (i.e. from acoustics, iridium, or wifi)?

[Alvaro Lorenzo]

Tricky. Maybe we want a field called something like platform_source: {USBL, Slocum, Ecosub, ALR ...} and other called transmission_mean : {Wifi, acoustics, iridium, starlink ... }

[Trishna Saeharaseelan]

To add warm-up time field in the planning config (e.g. for ecosub warm-up of sidescan payload)

[Dan Jones]

Needs the validator packages adding in here or in a requirements-dev.txt

[Trishna Saeharaseelan]

DONE

[Dan Jones]

Why aren't you validating the whole mock_data as a whole against the MESSAGE schema?

Should the tests be pytest/unittest or something so we can tell if they pass/fail in the CI later?

[Trishna Saeharaseelan]

1. Didn't just validate the FULL mock messages against message_schema because it kept passing even when it should have errors. Decided to break it down in the meantime to ensure it gets validated & does prompt errors (as it should). Added TODO in epic issue card to clean this up for better validation

2. DONE