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

manywheel: Refactor docker build / upload scripts #722

Merged
merged 4 commits into from
Apr 28, 2021

Conversation

seemethere
Copy link
Member

@seemethere seemethere commented Apr 27, 2021

Does a big refactor on the docker build / upload scripts to make them
more accessible similar to the build / upload process for conda and
libtorch

This also updates the README.md for manywheel/README.md

Enables #720 to be built easier

Copied from the README.md:

Building docker images

To build all docker images you can use the convenience script:

# Build without pushing
manywheel/build_all_docker.sh
# Build with pushing

To build a specific docker image use:

# GPU_ARCH_TYPE can be ["cuda", "rocm", "cpu"]
# GPU_ARCH_VERSION is GPU_ARCH_TYPE dependent, see manywheel/build_all_docker.sh for examples
GPU_ARCH_TYPE=cuda GPU_ARCH_VERSION=11.1 manywheel/build.sh

Signed-off-by: Eli Uriegas [email protected]

Does a big refactor on the docker build / upload scripts to make them
more accessible similar to the build / upload process for conda and
libtorch

This also updates the README.md for manywheel/README.md

Signed-off-by: Eli Uriegas <[email protected]>
@seemethere seemethere force-pushed the manywheel_docker_refactor branch from a28e713 to 5a4956f Compare April 27, 2021 23:49
manywheel/README.md Show resolved Hide resolved
manywheel/README.md Outdated Show resolved Hide resolved
seemethere and others added 2 commits April 28, 2021 09:50
Co-authored-by: Jane (Yuan) Xu <[email protected]>
Signed-off-by: Eli Uriegas <[email protected]>
@seemethere seemethere merged commit 6f4e0ca into pytorch:master Apr 28, 2021
@seemethere seemethere deleted the manywheel_docker_refactor branch April 28, 2021 16:52
@janeyx99 janeyx99 mentioned this pull request Apr 28, 2021
19 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants