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

pytorch multigrid updates #1620

Merged
merged 4 commits into from
Feb 26, 2021
Merged

pytorch multigrid updates #1620

merged 4 commits into from
Feb 26, 2021

Conversation

ECHO960
Copy link
Contributor

@ECHO960 ECHO960 commented Feb 20, 2021

(1) modify config logic for multigrid usage, now support (a) open long cycle and short cycle independently. (b) define long cycle by users
(2) rename multiGridSampler to multiGridHelper
(3) re-implement code for computing batch len in multi grid scenario.

Tested on AWS EC2 p3x16 machine

ECHO960 and others added 3 commits February 19, 2021 18:12
(1) modify config logic for multigrid usage, now support (a) open long cycle and short cycle independently. (b) define long cycle by users
(2) rename multiGridSampler to multiGridHelper
(3) re-implement code for computing batch len in multi grid scenario.
@ECHO960
Copy link
Contributor Author

ECHO960 commented Feb 23, 2021

hey, an error occurs when running detection cases, "MXNetError: cudaMalloc retry failed: out of memory". But I didn't change any code that related to detection, any ideas how to fix this?

@bryanyzhu
Copy link
Collaborator

@ECHO960 Sorry about the inconvenience, this is a bug in gluoncv-auto test, not related to your code. We temporarily disable this test in PR 1616 (#1616) and the PR was merged today. So please rebase with master and push again. This should solve this error. Thank you.

@github-actions
Copy link

Job PR-1620-4972b89 is done.
Docs are uploaded to http://gluon-vision-staging.s3-website-us-west-2.amazonaws.com/PR-1620/4972b89/index.html

Copy link
Collaborator

@bryanyzhu bryanyzhu left a comment

Choose a reason for hiding this comment

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

LGTM, thank you for the improvement.

@bryanyzhu bryanyzhu merged commit ec76b7d into dmlc:master Feb 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants