-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Docker builds for other CUDA versions, improve CI #4796
Conversation
|
||
AllenNLP releases Docker images to [Docker Hub](https://hub.docker.com/r/allennlp/) for each release. For information on how to run these releases, see [Installing using Docker](#installing-using-docker). | ||
|
||
### Building a Docker image |
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.
Why did this go away?
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.
I just moved it into the "Installing via Docker" section.
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.
You did 🙈
# Only run this for releases on the main repo, not forks. | ||
if: github.repository == 'allenai/allennlp' && github.event_name == 'release' | ||
# Only run this for releases. | ||
if: github.event_name == 'release' |
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.
Why did all of these if
statements change?
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.
We checked if github.repository == 'allenai/allennlp'
above, so it was redundant to have that again.
Feature request from our conversation here: #4793 (comment). This makes it so we'll build Docker images for both CUDA 10.2 and CUDA 11.0. These will be published as
allennlp/allennlp:v1.2.1-cuda10.2
andallennlp/allennlp:v1.2.1-cuda11.0
, respectively. We will still also provide alatest
tag which for now will default to the CUDA 10.2 image for consistency with PyTorch.This also merges our two GH Actions workflows (
.github/workflows/master.yml
and.github/workflows/pull_request.yml
) into a single one:.github/workflows/ci.yml
.I still have a couple things to do:
pull_request.yml
andmaster.yml
workflows. Could just call itci.yml
.