-
Notifications
You must be signed in to change notification settings - Fork 67
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
Vh 1216 fix spat plugin segfault simulation #621
Vh 1216 fix spat plugin segfault simulation #621
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.
There are some places where more error handling can be added especially while working with the clock. Suggest adding those in there.
Also, variable naming can be fixed to be consistent across.
|
||
## Introduction | ||
|
||
The SPaT Plugin is responsible for receiving information from the Traffic Signal Controller (TSC or SC) necessary for broadcasting Signal Phase and Timing (SPaT) messages. This includes querying any SNMP objects to determine TSC state and listen for any broadcast SPaT information from the TSC. |
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.
Is this received information
used to populate self-defined Ntcip1202
https://github.com/usdot-fhwa-OPS/V2X-Hub/blob/develop/src/v2i-hub/SpatPlugin/src/NTCIP1202.h? Can we add additional details about the received information
if (state == IvpPluginState_registered) { | ||
UpdateConfigSettings(); | ||
SpatPlugin::SpatPlugin(const std::string &name) :PluginClientClockAware(name) { | ||
spatReceiverThread = std::make_unique<tmx::utils::ThreadTimer>(std::chrono::milliseconds(5)); |
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.
Is there a reason we choose 5 milliseconds?
Quality Gate failedFailed conditions |
PR Details
Description
This pull request updates the SPAT plugin to fix a number of issues with the legacy implementation.
Additionally to help provide these improvements in the SPaT plugin, several improvements were made in the TMX Utils functionality.
Related Issue
VH-1216
Motivation and Context
Fix SPaT plugin for simulation and improve overall usability and maintainability.
How Has This Been Tested?
Tested using virtual signal controller for both binary and hex spat payloads.
Also tested in sim mode using the following steps
src/v2i-hub/CDASimAdapter/scripts/send_timestep_udp.py
script to send periodic time sync messages.Types of changes
Checklist:
V2XHUB Contributing Guide