-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Simulator: update MetaDrive to latest #32576
Conversation
Thanks for contributing to openpilot! In order for us to review your PR as quickly as possible, check the following:
|
@@ -140,7 +140,7 @@ inputs = "*" | |||
Jinja2 = "*" | |||
lru-dict = "*" | |||
matplotlib = "*" | |||
metadrive-simulator = { version = "0.4.2.3", markers = "platform_machine != 'aarch64'" } # no linux/aarch64 wheels for certain dependencies | |||
metadrive-simulator = {git = "https://github.com/metadriverse/metadrive.git", rev ="233a3a1698be7038ec3dd050ca10b547b4b3324c", markers = "platform_machine != 'aarch64'" } # no linux/aarch64 wheels for certain dependencies |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we just unpin it? have they done a release since that fix? I don't remember why we pinned it in the first place
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unpinned. 0.4.2.3 is the last they did and that was before the fix
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh then a git commit is fine if there's no release. just leave a comment to explain why so we can change it later
76a7e02
to
09ae682
Compare
* bump metadrive * comment old-commit-hash: aa081f5
prereq for #32513.
this includes this fix: metadriverse/metadrive#632