-
Notifications
You must be signed in to change notification settings - Fork 6
Added test for operational intent submission #76
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds support for submitting operational intents directly to Flight Blender, implementing a test scenario for this new functionality alongside the necessary models, client methods, and configuration updates. The implementation includes dependency upgrades to newer versions of implicitdict (3.0.0 → 4.0.1) and uas-standards (3.5.0 → 4.2.0) to support ASTM F3548 Volume4D data structures.
Key Changes:
- Added new test scenario
test_add_operational_intent.pyfor operational intent submission - Introduced models for operational intent declarations including enums for operation states and types
- Updated Flight Blender client with methods to handle operational intent uploads and setup
Reviewed changes
Copilot reviewed 15 out of 16 changed files in this pull request and generated 11 comments.
Show a summary per file
| File | Description |
|---|---|
| uv.lock | Updated dependency versions for implicitdict and uas-standards |
| pyproject.toml | Updated dependency versions in project configuration |
| src/openutm_verification/simulator/models/declaration_models.py | Added enums and models for operational intent declarations |
| src/openutm_verification/simulator/flight_declaration.py | Implemented Volume4D generation and operational intent declaration method |
| src/openutm_verification/scenarios/test_add_operational_intent.py | New test scenario for operational intent submission |
| src/openutm_verification/scenarios/registry.py | Added support for operational intent data in scenario results |
| src/openutm_verification/scenarios/common.py | Added helper function to generate operational intent declarations |
| src/openutm_verification/core/reporting/reporting_models.py | Extended ScenarioResult to include operational intent data |
| src/openutm_verification/core/execution/scenario_runner.py | Added context management for operational intent declarations |
| src/openutm_verification/core/execution/dependencies.py | Updated dependency injection to handle operational intent config |
| src/openutm_verification/core/execution/config_models.py | Added configuration field for operational intent declarations |
| src/openutm_verification/core/clients/flight_blender/flight_blender_client.py | Implemented upload and setup methods for operational intent |
| src/openutm_verification/assets/simulator_templates/flight_declaration_via_operational_intent_template.json | New template for operational intent declarations |
| src/openutm_verification/assets/simulator_templates/flight_declaration_template.json | Modified existing template (contains breaking change) |
| config/default.yaml | Added configuration path for operational intent declarations |
| config/bern/flight_declaration_via_operational_intent.json | New configuration file with bounding box for operational intent |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
src/openutm_verification/assets/simulator_templates/flight_declaration_template.json
Outdated
Show resolved
Hide resolved
src/openutm_verification/core/clients/flight_blender/flight_blender_client.py
Outdated
Show resolved
Hide resolved
src/openutm_verification/scenarios/test_add_operational_intent.py
Outdated
Show resolved
Hide resolved
src/openutm_verification/simulator/models/declaration_models.py
Outdated
Show resolved
Hide resolved
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…laration_template.json Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ender_client.py Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This PR adds a test to verify the behaviour when submitting an operational intent directly