-
Notifications
You must be signed in to change notification settings - Fork 11
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
Voices/vug pilot2 #341
base: develop
Are you sure you want to change the base?
Voices/vug pilot2 #341
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.
Lets remove all the files that are not needed for voices.
For example, I think the whole cdasim_config folder can be moved right?
And it'd be great if we could give some high level documentation about what the scenario is about and replace the ones in docs
it'd be good if we could include osm map, MAP uper, or evc controller configurations so we have a version control. *As long as it is allowed for us to publish them
# Defines the ros parameters which define the characteristics of this host vehicle configuration | ||
# The value type field is used to indicate how the field should be set. | ||
# Should it be treated as a measured value (Measured) or a desired behavior constraint (Desired) | ||
use_sim_time: true |
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.
the wall time argument is missing. Could you also add the parameter to vru yaml file and set it false
since it doesnt use wall time in its spat?
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.
Just added this
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.
@adev4a
please remember to add to ail_vru_uc1_scenario
folder and set it false
otherwise cdasim would break down. I would even add to xil_cloud and telematics ones as well.
@ajloughr-leidos can you look into this? |
# Defines the ros parameters which define the characteristics of this host vehicle configuration | ||
# The value type field is used to indicate how the field should be set. | ||
# Should it be treated as a measured value (Measured) or a desired behavior constraint (Desired) | ||
use_sim_time: true |
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.
@adev4a
please remember to add to ail_vru_uc1_scenario
folder and set it false
otherwise cdasim would break down. I would even add to xil_cloud and telematics ones as well.
PR Details
Adds config params required for voices.
Description
The VOICES project uses the CARMA Platform with the CARMA CARLA integration, but does not use the rest of CDA Sim. A new config folder was added with minor changes and tuning specifically for this use and the Mcity CARLA map.
Related Issue
Motivation and Context
How Has This Been Tested?
Types of changes
Checklist:
CARMA Contributing Guide