Bugfix: incorrect BACKBONE_OUT_CHANNELS due to duplicate RESNET keys in yaml files

[bernhardschaefer]

In some YAML files the RESNETS key appears twice.

yacs uses pyyaml.safe_load(), which considers only the last key occurrence.

In the adapted configs, this means that instead of the specified 256 channels the models were actually trained with the default 1024 channels.

from maskrcnn_benchmark.config import cfg
print(cfg.MODEL.RESNETS.BACKBONE_OUT_CHANNELS)  # -> 1024

cfg.merge_from_file("../configs/quick_schedules/e2e_mask_rcnn_X_101_32x8d_FPN_quick.yaml")

print(cfg.MODEL.RESNETS.BACKBONE_OUT_CHANNELS) # -> still 1024 instead of 256

Imho yacs should throw an error in case of duplicate keys, as requested in an open bug report in pyyaml: yaml/pyyaml#165
I will create an issue within yacs to point to that issue.

[fmassa]

Thanks for the bugfix! This was due to a bad rebase on my side :-/