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

AWSIM <-> ss2 connection traffic lights support #230

Merged
merged 15 commits into from
Dec 25, 2023

Conversation

piotr-zyskowski-rai
Copy link
Contributor

@piotr-zyskowski-rai piotr-zyskowski-rai commented Dec 13, 2023

PR description

This PR includes the following changes:

  • Traffic lights support for AWSIM <-> ss2 connection
    • traffic light states are received by AWSIM via the ZMQ interface and appropriate traffic signals are lit
  • Documentation on:
    • How to run a sample traffic lights scenarios (both conventional and V2I)
    • How to prepare scenarios to work with AWSIM including the limitations of the scenarios
    • How to prepare the AWSIM scene to work with scenario simulation

Other context

This PR includes a fix for RGL deinitalization #231. It is required to despawn and spawn Ego vehicle again within the same simulation

@piotr-zyskowski-rai piotr-zyskowski-rai marked this pull request as ready for review December 14, 2023 18:38
@piotr-zyskowski-rai
Copy link
Contributor Author

The link checker failed due to release links which will be ready after the release is finalized I believe.

✗ [404] https://github.com/tier4/AWSIM/releases/download/v1.2.0/AWSIM_v1.2.0.zip | Network error: Not Found
✗ [404] https://github.com/tier4/AWSIM/releases/download/v1.2.0/shinjuku_map.zip | Network error: Not Found
✗ [404] https://github.com/tier4/AWSIM/releases/download/v1.2.0/AWSIM_v1.2.0.zip | Network error: Not Found
✗ [404] https://github.com/tier4/AWSIM/releases/download/v1.2.0/AWSIM_v1.2.0_ss2.zip | Network error: Not Found

Copy link
Collaborator

@mackierx111 mackierx111 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

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.

None yet

3 participants