-
Notifications
You must be signed in to change notification settings - Fork 9.1k
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
Minimize MetaDrive install size #33083
Labels
Comments
adeebshihadeh
added
bounty
cleanup
development
related to the openpilot development experience
and removed
bounty
labels
Jul 26, 2024
Most of panda3d are shared libraries. I did a quick test and for our current use case (the bridge test) we don't need:
those 3 alone are 33% of panda3d size |
So I was able to reduce the size of metadrive to 53M and ran it here #33085 Steps:
|
From
|
maxime-desroches
moved this from Todo
to In Progress
in Improving the openpilot development experience
Aug 16, 2024
Merged
github-project-automation
bot
moved this from In Progress
to Done
in Improving the openpilot development experience
Sep 4, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
MetaDrive is going to be a core part of the openpilot on PC experience soon, so most openpilot installations will want to have it installed. Since openpilot itself is small, this means MetaDrive is a significant percent of an openpilot install.
MetaDrive and its two of its dependencies, panda3D and cv2, are our top 3 largest dependencies combining for 691M.
In order to claim the bounty, make our MetaDrive install size as small as possible, including:
panda3D
any smaller? if so, let's do the same thing that we're doing for MetaDriveMetaDrive PRs should be made against upstream MetaDrive, however if they don't respond quickly, we can merge them into our fork.
For validation, we have a CI test that should mostly cover what we care about:
openpilot/.github/workflows/tools_tests.yaml
Line 28 in d128dbe
The text was updated successfully, but these errors were encountered: