-
Notifications
You must be signed in to change notification settings - Fork 13
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
Feature/vru two heavy vehicles #172
Conversation
<?xml version="1.0" encoding="UTF-8"?> | ||
|
||
<!-- generated on 2023-11-06 15:52:06 by Eclipse SUMO netedit Version 1.18.0 | ||
--> | ||
|
||
<additional xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="http://sumo.dlr.de/xsd/additional_file.xsd"> | ||
<!-- StoppingPlaces --> | ||
<busStop id="bs_2" lane="10_3" startPos="2.00" endPos="8.00"/> | ||
</additional> |
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.
If we are not using bus stop, do we still need this? or is also being used for the heavy_vehicles to stop?
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.
Yes, it's actually used in the rou.xml file, under the third car
<stop busStop="bs_2" duration="5.00"/> | ||
</vehicle> | ||
<vehicle id="v_3" type="vehicle.carlamotors.carlacola" depart="999.00" route="r_0"/> |
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.
what is this one? seems like there are 4 vehicles?
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.
This one is used for keeping the scenario alive, if we only have three cars, the simulation will stop after the last car finishes, might not give us time for other Carma stuff we want to do, this is a safety
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.
Can we add some sort of comment indicating that this vehicle is just there to extend the CARLA scenario.
co-simulation/bundle/src/assembly/resources/scenarios/Town04/sumo/Town04.sumocfg
Show resolved
Hide resolved
co-simulation/bundle/src/assembly/resources/scenarios/Town04/sumo/Town04.sumocfg
Show resolved
Hide resolved
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information The version of Java (11.0.21) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17. |
PR Details
Description
add 3 trucks for VRU scenario
Related GitHub Issue
Related Jira Key
CDAR-432
Motivation and Context
How Has This Been Tested?
Types of changes
Checklist: