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

fix dockerfile, add auto-launch script #1513

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ies0411
Copy link

@ies0411 ies0411 commented Nov 13, 2023

Hello.

The existing dockerfile has several problems.

  • Dockerfile : too old to work.
  • cu116.Dockerfile : It takes so much time to build docker image. If you launch Dynamic VFE, it emits an error about torch-scatter and kornia.

So, I suggest new dockerfile and script ( auto script for making docker image and launching docker image easily)
The dockerfile is lighter than the before version (since it contains an essential library for implementing openpcdet) and doesn't have an error about torch-scatter.

I want to openpcdet contributor continuously :)
If there are some error or issues, I will fix ASAP.
Thank you !!

@ies0411 ies0411 marked this pull request as draft May 14, 2024 00:41
@ies0411 ies0411 marked this pull request as ready for review May 14, 2024 00:42
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.

1 participant