[MMSIG] Support mmdeploy Docker for Jetson #2587
Conversation
|
I will add the other file later |
Create Jetson_docker.md
|
Hi, @RunningLeon where can I add the Jetson docker CI, and can I make an individual CI file for Jetson docker？ |
build PyTorch 1.10.0 with python3.8 build torchvision 0.11.0 with python3.8
solve the problem 3 && 4 using sed in docker
| python3.8 -m pip install --upgrade --no-cache-dir setuptools packaging 'Cython<3' wheel &&\ | ||
| python3.8 -m pip install --no-cache-dir --verbose wget psutil numpy &&\ | ||
| python3.8 -m pip install --upgrade --force-reinstall --no-cache-dir --verbose cmake protobuf | ||
| python3.8 -m pip install onnx==1.10 versioned-hdf5 numpy |
There was a problem hiding this comment.
why install versioned-hdf5 numpy here?
There was a problem hiding this comment.
numpy here need to update, versioned-hdf5 here for pycuda
| apt-get install -y vim wget libspdlog-dev libssl-dev libpng-dev pkg-config libhdf5-100 libhdf5-dev patch --no-install-recommends\ | ||
| python3.8 python3.8-dev python3.8-pip --no-install-recommends &&\ | ||
| python3.8 -m pip install --upgrade --no-cache-dir setuptools packaging 'Cython<3' wheel &&\ | ||
| python3.8 -m pip install --no-cache-dir --verbose wget psutil numpy &&\ |
There was a problem hiding this comment.
why need to install these packages?
There was a problem hiding this comment.
python3.8 -m pip install --upgrade --no-cache-dir setuptools packaging 'Cython<3' wheel &&\ python3.8 -m pip install --no-cache-dir --verbose wget psutil numpy &&\
this package for build the pytorch, I referenc from l4t repo
| RUN apt-key adv --keyserver keyserver.ubuntu.com --recv-keys 42D5A192B819C5DA &&\ | ||
| apt-get remove python3 &&\ | ||
| apt-get update &&\ | ||
| apt-get install -y vim wget libspdlog-dev libssl-dev libpng-dev pkg-config libhdf5-100 libhdf5-dev patch --no-install-recommends\ |
There was a problem hiding this comment.
why do we need libhdf5-100 libhdf5-dev patch?
There was a problem hiding this comment.
https://github.com/open-mmlab/mmdeploy/blob/main/docs/zh_cn/01-how-to-build/jetsons.md
reference from install h5py and pycuda
docker/Jetson/Jetpack5/Dockerfile
Outdated
| make -j$(nproc) && make install | ||
|
|
||
| # add the patch to solve the torch.distributed issue in docker | ||
| RUN cd mmdeploy/tools &&\ |
There was a problem hiding this comment.
could we use rewrite to do this ?
There was a problem hiding this comment.
Hi, I have no idea how to rewrite this TAT.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2587 +/- ##
==========================================
- Coverage 48.50% 48.36% -0.14%
==========================================
Files 355 358 +3
Lines 13795 13857 +62
Branches 2050 2048 -2
==========================================
+ Hits 6691 6702 +11
- Misses 6653 6700 +47
- Partials 451 455 +4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily receiving feedbacks. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Install mmdeploy on Jetson is really hard, so build a docker may a gd
Modification
Please briefly describe what modification is made in this PR.
BC-breaking (Optional)
Does the modification introduce changes that break the backward-compatibility of the downstream repositories?
no
Use cases (Optional)
see jetson_docker.md .
Checklist