rbgirshick/yacs

merge_from_list ignores 'new_allowed' option

theseafaringturtle opened this issue · 0 comments

I was quite puzzled to find out that set_new_allowed worked as expected when merging a config file, but had no effect when merging a list.
Looking at the code, it seems merge_from_list lacks a check for that option.

I'd be happy to create a PR if you agree that this should be changed.