support specifying matching_heuristics from filename

[wat3rBro]

Summary: D35519683 changed the default matching_heuristics from False to True, it may cause some issues:

• potential bug report: https://fb.workplace.com/groups/2240361332735959/posts/5264606250311437/?comment_id=5265167493588646
• bloated logging: https://github.com/facebookresearch/detectron2/issues/4364

This diff reverts the default value back to False, and introduce a way to specify this option via filename url.

Differential Revision: D41145857

[facebook-github-bot]

This pull request was exported from Phabricator. Differential Revision: D41145857

[facebook-github-bot]

This pull request was exported from Phabricator. Differential Revision: D41145857

[facebook-github-bot]

This pull request was exported from Phabricator. Differential Revision: D41145857

[ppwwyyxx]

Thanks a lot for doing this! I think you'd have to bump the version number and let detectron2 depend on this version of fvcore.

Also, new versions of fvcore get published to pypi automatically through circleci, only if cpu/gpu tests passed.. Unfortunately, tests have been broken for many months, so that's another dependency.. :cry:

[ppwwyyxx]

An alternative is to overwrite .load() in detectron2 so you can leave fvcore as is

[facebook-github-bot]

This pull request was exported from Phabricator. Differential Revision: D41145857

[facebook-github-bot]

This pull request was exported from Phabricator. Differential Revision: D41145857

[facebook-github-bot]

This pull request was exported from Phabricator. Differential Revision: D41145857

[facebook-github-bot]

This pull request has been merged in facebookresearch/fvcore@d194d76a30b634cc200195252975df3a0eb98a67.