[WIP Patched] Cudnn grouped convolution nhwc patch#31633
[WIP Patched] Cudnn grouped convolution nhwc patch#31633VitalyFedyunin wants to merge 1 commit intogh/VitalyFedyunin/104/basefrom
Conversation
Earlier cudnn version doesn't support grouped convolution in NHWC well. Legit configuration in later cudnn version might return CUDNN_STATUS_NOT_SUPPORTED. We are falling back to NCHW when runtime check of cudnn version is < 7.6.0 to keep the logic simple. Note: We might update the heuristics, 7.6.0 is very conservative. [ghstack-poisoned]
Earlier cudnn version doesn't support grouped convolution in NHWC well. Legit configuration in later cudnn version might return CUDNN_STATUS_NOT_SUPPORTED. We are falling back to NCHW when runtime check of cudnn version is < 7.6.0 to keep the logic simple. Note: We might update the heuristics, 7.6.0 is very conservative. ghstack-source-id: a16ae00 Pull Request resolved: #31633
💊 CircleCI build failures summary and remediationsAs of commit 09eae89:
Detailed failure analysisOne may explore the probable reasons each build failed interactively on the Dr. CI website. 🕵️ 13 new failures recognized by patternsThe following build failures do not appear to be due to upstream breakage:
|
Stack from ghstack:
zeros,ones,full#31131 Add memory_format support tozeros,ones,fullEarlier cudnn version doesn't support grouped convolution in NHWC well. Legit
configuration in later cudnn version might return CUDNN_STATUS_NOT_SUPPORTED.
We are falling back to NCHW when runtime check of cudnn version is < 7.6.0 to
keep the logic simple.
Note:
We might update the heuristics, 7.6.0 is very conservative.