Skip to content

feat(planning_data_analyzer): add open-loop drivable area compliance metric#381

Merged
beomseok-kimm merged 3 commits intoautowarefoundation:mainfrom
tier4:feat/epdms-dac
Mar 31, 2026
Merged

feat(planning_data_analyzer): add open-loop drivable area compliance metric#381
beomseok-kimm merged 3 commits intoautowarefoundation:mainfrom
tier4:feat/epdms-dac

Conversation

@beomseok-kimm
Copy link
Copy Markdown
Contributor

@beomseok-kimm beomseok-kimm commented Mar 30, 2026

Description

This PR is the second implementation step in a series toward full EPDMS support in autoware_planning_data_analyzer, following the NAVSIM metric definition and reference implementation.

This PR specifically adds DAC (Drivable Area Compliance) as the next standalone subscore.

How was this PR tested?

Successfully created output rosbag and JSON results that contain the DAC metric when route, vector map, and vehicle info are available.

input and output link:

Notes for reviewers

The current DAC implementation is a package-level approximation based on preferred lanelets and ego footprint corner checks.

Effects on system behavior

None.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 30, 2026

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

Copy link
Copy Markdown
Contributor

@go-sakayori go-sakayori left a comment

Choose a reason for hiding this comment

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

Have you tested with DLR? I think some changes are needed on DLR too

Comment thread planning/autoware_planning_data_analyzer/launch/planning_data_analyzer.launch.xml Outdated
Comment thread planning/autoware_planning_data_analyzer/launch/planning_data_analyzer.launch.xml Outdated
Comment thread planning/autoware_planning_data_analyzer/launch/planning_data_analyzer.launch.xml Outdated
Comment thread planning/autoware_planning_data_analyzer/launch/planning_data_analyzer.launch.xml Outdated
Comment thread planning/autoware_planning_data_analyzer/src/metrics/trajectory_metrics.cpp Outdated
Comment thread planning/autoware_planning_data_analyzer/src/metrics/drivable_area_compliance.cpp Outdated
Comment thread planning/autoware_planning_data_analyzer/src/metrics/drivable_area_compliance.cpp Outdated
@beomseok-kimm
Copy link
Copy Markdown
Contributor Author

beomseok-kimm commented Mar 30, 2026

Updated in ee58930. I also revalidated the analyzer through the open-loop pipeline on local bags that include route + /map/vector_map and explicit vehicle_model input.

@beomseok-kimm
Copy link
Copy Markdown
Contributor Author

Have you tested with DLR? I think some changes are needed on DLR too

I validated the analyzer changes through the current open-loop pipeline and local bags, but I have not yet done a full DLR package-level compatibility sweep. Since this PR changes analyzer launch/output behavior, DLR integration is a valid concern. If needed, I will do a follow-up DLR compatibility check after the EPDMS implementation series.

@go-sakayori
Copy link
Copy Markdown
Contributor

Since this PR changes analyzer launch/output behavior, DLR integration is a valid concern.

The evaluator inside TIER IV uses the DLR, so better to check ASAP. I think it won'T be a problem for now, since we allow the vehicle model to be empty.

@beomseok-kimm
Copy link
Copy Markdown
Contributor Author

@go-sakayori
I pached the dlr a bit , adding vehicle info argument to make it compatible with DAC!
tier4/driving_log_replayer_v2#344

@go-sakayori
Copy link
Copy Markdown
Contributor

@beomseok-kimm
Can you deal with the unresolved comments?

@beomseok-kimm beomseok-kimm merged commit 77e634e into autowarefoundation:main Mar 31, 2026
29 of 30 checks passed
@beomseok-kimm beomseok-kimm deleted the feat/epdms-dac branch March 31, 2026 04:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants