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.

Sign up for GitHub

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

Jump to bottom

Vehicle gap metric #1971

Merged
merged 40 commits into from
Apr 22, 2023
Merged

Vehicle gap metric #1971

merged 40 commits into from
Apr 22, 2023

Conversation

Adaickalavan
Copy link
Member

@Adaickalavan Adaickalavan commented Apr 19, 2023 •
edited
Loading

  • Introduced new gap-between-vehicles cost function in metric module.
  • Renamed GapBetweenVehicles cost to VehicleGap cost in metric module.
  • This PR is to be merged in the following order

Adaickalavan and others added 30 commits April 10, 2023 11:03
@Adaickalavan
Add driving-smarts-2023
3fa2f81
@Adaickalavan
Add scenario attribute.
83a9a0f
@Adaickalavan
Merge branch 'master' into driving-smarts-2023
0369996
@Adaickalavan
Add changelog.
4df4c4e
@Adaickalavan
Modify metrics.
218b035
@Adaickalavan
Add changelog.
dc1951a
@Adaickalavan
Merge branch 'master' into new-metrics
9db4070
@Adaickalavan
Merge branch 'master' into new-metrics
9dce0be
@Adaickalavan
Merge branch 'master' into new-metrics
703db59
@Adaickalavan @Gamenot
Update CHANGELOG.md
aac6333 
Co-authored-by: Tucker Alban <[email protected]>
@Adaickalavan
Remove regexp for actors of interest.
e543157
@Adaickalavan @Gamenot
Update CHANGELOG.md
00ddc47 
Co-authored-by: Tucker Alban <[email protected]>
@Adaickalavan
Rephrased changelog.
4881671
@Adaickalavan
Activate comfort cost.
011d519
@Adaickalavan
Add comfort metric.
1df89b5
@Adaickalavan
Edit comments.
46e3a16
@Adaickalavan
Merge branch 'master' into metric-comfort
9dcd950
@Adaickalavan
Merge branch 'master' into new-metrics
151b66a
@Adaickalavan
Edit changelog.
7fa5ece
@Adaickalavan
Merge branch 'new-metrics' into metric-comfort
3af2bf3
@Adaickalavan
Merge branch 'master' into new-metrics
6c0a384
@Adaickalavan
Merge branch 'new-metrics' into metric-comfort
a15f946
@Adaickalavan @saulfield
Remove excess whitespace.
e955837 
Co-authored-by: Saul Field <[email protected]>
@Adaickalavan
Add gap metric.
09ac1c0
@Adaickalavan
Add gap metric.
a7d1602
@Adaickalavan
Add gap metric.
ebce109
@Adaickalavan
Renamed gap_between_vehicles to vehicle_gap.
51e6871
@Adaickalavan
gap metric : midway commit.
b695b94
@Adaickalavan
Merge branch 'master' into metric-average-gap
34196db
@Adaickalavan
Draft gap metric.
3fe268b
@Adaickalavan Adaickalavan self-assigned this Apr 19, 2023
Base automatically changed from metric-comfort to master April 21, 2023 23:08
@Adaickalavan Adaickalavan merged commit 2b1b0af into master Apr 22, 2023
@Adaickalavan Adaickalavan deleted the metric-average-gap branch April 22, 2023 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Gamenot Gamenot Gamenot approved these changes

@saulfield saulfield Awaiting requested review from saulfield

@qianyi-sun qianyi-sun Awaiting requested review from qianyi-sun

Assignees

@Adaickalavan Adaickalavan

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Adaickalavan @Gamenot