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

feat(map_loader): show intersection areas #5401

Merged

Conversation

satoshi-ota
Copy link
Contributor

@satoshi-ota satoshi-ota commented Oct 25, 2023

Description

Show intersection area polygon.

related PR:

Screenshot from 2023-10-25 09-07-16

Tests performed

Psim.

Effects on system behavior

Nothing.

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@Motsu-san
Copy link
Contributor

@satoshi-ota Thank you for your PR! Could you tell us who you want to review this?

@satoshi-ota
Copy link
Contributor Author

@satoshi-ota Thank you for your PR! Could you tell us who you want to review this?

@Motsu-san Thanks! Yes, I would like you to review this PR. (On the other hand, I asked @mitsudome-r san to review this in slack.)

@Motsu-san
Copy link
Contributor

@satoshi-ota The following PR is related isn't it?
autowarefoundation/autoware_launch#655

@satoshi-ota
Copy link
Contributor Author

@satoshi-ota The following PR is related isn't it? autowarefoundation/autoware_launch#655

@Motsu-san YES! I'll add link to PR description.

@satoshi-ota satoshi-ota added the run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) label Oct 25, 2023
Motsu-san
Motsu-san approved these changes Oct 25, 2023
@Motsu-san Motsu-san self-requested a review October 25, 2023 01:57
@Motsu-san
Copy link
Contributor

Sorry for my confusing LGTM post, I will approve this after all checks are passed.

@mitsudome-r
Copy link
Member

FYI, I have merged the dependent PR in autoware_common.

@satoshi-ota
Copy link
Contributor Author

Conflict... 🥺 I'll fix it.

@satoshi-ota satoshi-ota force-pushed the feat/show-intersection-area branch from dce9649 to 8a3829c Compare October 25, 2023 08:44
@satoshi-ota satoshi-ota enabled auto-merge (squash) October 25, 2023 08:45
@satoshi-ota satoshi-ota merged commit 4eb0d45 into autowarefoundation:main Oct 25, 2023
@satoshi-ota satoshi-ota deleted the feat/show-intersection-area branch October 26, 2023 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:map Map creation, storage, and loading. (auto-assigned) run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants