facebookresearch / facebookresearch/detectron2

DensePose DeepLabHead setting is wired.

Open
#1,995 1 comment 0 reactions 0 assignees View on GitHub
densepose
Dominant language
Python
Stars
34.7k
Forks
7.9k
PR merge metrics
No merged PRs in 30d

Description

In [DensePoseDeepLabHead](https://github.com/facebookresearch/detectron2/blob/729290df8a1b9c0307fb667dd776bed3c4f8bdc5/projects/DensePose/densepose/densepose_head.py#L98), you set atrous_rates in [ASSP](https://github.com/facebookresearch/detectron2/blob/729290df8a1b9c0307fb667dd776bed3c4f8bdc5/projects/DensePose/densepose/densepose_head.py#L185) [6, 12, 56], while in [DeepLabv3](https://arxiv.org/pdf/1706.05587.pdf) atrous_rates is [6, 12, 18] and in your copied link [torchvision](https://github.com/pytorch/vision/blob/master/torchvision/models/segmentation/deeplabv3.py), the atrous_rates is [12, 24, 36] which is double to [6, 12, 24]. Are your settings designed for DensePose task or some other reasons?

Meanwhile, you replace batch normalization by group normalization, while deeplabv3 paper and torchvision code both use batch normalization. Is there some reason to use group normalization? Maybe you make some experiments which prove GN is better?

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.