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

Release candiadate #2310

Merged
merged 74 commits into from
Dec 21, 2019
Merged

Release candiadate #2310

merged 74 commits into from
Dec 21, 2019

Conversation

bernatx
Copy link
Contributor

@bernatx bernatx commented Dec 20, 2019

Description

WIP: This is a first candidate version for 0.9.7

Where has this been tested?

  • Platform(s): Ubuntu, Windows
  • Python version(s): 2.7, 3.7
  • Unreal Engine version(s): 4.22

Possible Drawbacks

Some problems are detected at the traffic lights in some special cases, where two fast vehicles could enter the traffic light, and only the first one will be notified of the traffic light.


This change is Reviewable

jackbart94 and others added 30 commits December 12, 2019 16:11
Removed redundant fps cap in BatchControl
Tuned PID values for synchronised mode
Fixed buffer data passed from localization stage to collision stage
to localization stage

Temporary patch for ignoring unregistered actors during lane change

Fixes for geodesic gridding logic

Added exception handling logic to PipelineStage class

Setting simulator back to variable time step asyncrhonous mode
in TrafficManager's destructor
to work in synchronous execution
Implemented geodesic grid centering logic
Modified messenger class to a blocking queue implementation
Fixed SnippetProfiler to work on multiple snippet
Using larger waypoint grids for pedestrian localization
Removed snippet profiling code from localization stage

(cherry picked from commit 4192906)
(cherry picked from commit db0516c)
@bernatx bernatx added this to the 0.9.7 milestone Dec 20, 2019
@bernatx bernatx self-assigned this Dec 20, 2019
@update-docs
Copy link

update-docs bot commented Dec 20, 2019

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would update our CHANGELOG.md based on your changes.

Copy link
Contributor

@marcgpuig marcgpuig left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 41 of 41 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @jackbart94)

Copy link
Contributor

@jackbart94 jackbart94 left a comment

Choose a reason for hiding this comment

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

Reviewed 41 of 41 files at r1.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

Copy link
Contributor

@jackbart94 jackbart94 left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: :shipit: complete! all files reviewed, all discussions resolved

@marcgpuig marcgpuig merged commit b9fbbf7 into master Dec 21, 2019
@delete-merged-branch delete-merged-branch bot deleted the release_candiadate branch December 21, 2019 03:59
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.

6 participants