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

Geo Message Query Updates & Transfer to Processed BSM Data #36

Merged

Conversation

dmccoystephenson
Copy link
Collaborator

@dmccoystephenson dmccoystephenson commented Feb 4, 2025

PR Details

Description

This PR includes the following changes to the Geo Message Query capability in the CV Manager:

  1. Uses ProcessedBsm and ProcessedPsm collections to query for geospatial data
  2. Fixes broken UI behavior with the V2x Msg Viewer layer
  3. Adds polygon & point previews to give the user more feedback
  4. Uses different colors for BSMs/PSMs with different values for properties.id
  5. Removed the geo_msg_query microservice.

How Has This Been Tested?

The changes have been tested through unit tests and integration tests. The unit tests for the relevant components were updated and executed using the existing testing methods. The integration test involved the following steps:

  1. Adding specific environment variables to the configuration file.
  2. Rebuilding the containers to include the latest updates.
  3. Starting the containers to ensure the environment is set up correctly.
  4. Interacting with the application through the UI by performing relevant actions and verifying the functionality.
  5. Selecting default settings and submitting data.
  6. Observing the visualized output to ensure the changes behave as expected.

These steps were carried out to ensure that the changes integrate smoothly with the existing system and do not introduce unexpected behavior.

Types of changes

  • Defect fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that cause existing functionality to change)

Checklist:

  • My changes require new environment variables:
    • I have updated the docker-compose, K8s YAML, and all dependent deployment configuration files.
  • My changes require updates to the documentation:
    • I have updated the documentation accordingly.
  • My changes require updates and/or additions to the unit tests:
    • I have modified/added tests to cover my changes.
  • All existing tests pass.

Michael7371 and others added 30 commits January 21, 2025 15:21
Synchronize Map Menu Bug Fixes From Release Branch
Michael7371 and others added 4 commits February 4, 2025 11:50
Co-authored-by: jacob6838 <51723791+jacob6838@users.noreply.github.com>
Geo Message Query Updates & Transfer to Processed BSM Data
@dmccoystephenson dmccoystephenson marked this pull request as ready for review February 4, 2025 22:51
@dmccoystephenson dmccoystephenson marked this pull request as draft February 5, 2025 15:37
@dmccoystephenson dmccoystephenson marked this pull request as ready for review February 5, 2025 20:14
@SaikrishnaBairamoni SaikrishnaBairamoni merged commit 8bf5c67 into usdot-jpo-ode:release/2025-q1 Feb 5, 2025
4 of 5 checks passed
@drewjj drewjj deleted the cdot-release-2025-q1 branch February 7, 2025 18:18
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.

4 participants