Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

update dockerfile according to the new INSTALL.md #883

Merged
merged 1 commit into from
Jun 12, 2019
Merged

update dockerfile according to the new INSTALL.md #883

merged 1 commit into from
Jun 12, 2019

Conversation

keineahnung2345
Copy link
Contributor

From https://github.com/facebookresearch/maskrcnn-benchmark/blob/master/INSTALL.md:

  1. additionally install tqdm, cudatoolkit, apex
  2. add FORCE_CUDA argument

Note that the argument CUDA is re-declared, this is done so that its value can be reused, following the instruction here.

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Jun 12, 2019
@fmassa
Copy link
Contributor

fmassa commented Jun 12, 2019

cc @miguelvr

@miguelvr
Copy link
Contributor

LGTM 👍

Copy link
Contributor

@fmassa fmassa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@fmassa fmassa merged commit d173a85 into facebookresearch:master Jun 12, 2019
@keineahnung2345 keineahnung2345 deleted the dockerfile-201906 branch April 30, 2020 13:05
Lyears pushed a commit to Lyears/maskrcnn-benchmark that referenced this pull request Jun 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed Do not delete this pull request or issue due to inactivity.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants