Skip to content

Conversation

aivanou
Copy link
Contributor

@aivanou aivanou commented Oct 21, 2021

Summary:
Change lightning-cv example to make it work for gpu training
It seems there are bugs with ddp2 running on GPUs, so changing accelerator to ddp

Reviewed By: d4l3k

Differential Revision: D31818113

Summary:
Change `lightning-cv` example to make it work for gpu training
It seems there are bugs with `ddp2` running on GPUs, so changing accelerator to `ddp`

Reviewed By: d4l3k

Differential Revision: D31818113

fbshipit-source-id: 2f72166af21d5f364d61c48ab84e14920609a9cf
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Oct 21, 2021
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D31818113

@codecov
Copy link

codecov bot commented Oct 21, 2021

Codecov Report

Merging #293 (dfd7614) into main (851745e) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #293   +/-   ##
=======================================
  Coverage   94.71%   94.71%           
=======================================
  Files          56       56           
  Lines        2670     2670           
=======================================
  Hits         2529     2529           
  Misses        141      141           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 851745e...dfd7614. Read the comment docs.

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in a7727c7.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants