Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

allow test for any checkpoint #874

Merged
merged 1 commit into from
Jun 12, 2019
Merged

allow test for any checkpoint #874

merged 1 commit into from
Jun 12, 2019

Conversation

wangg12
Copy link
Contributor

@wangg12 wangg12 commented Jun 10, 2019 •
edited
Loading

Regarding this issue #819

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Jun 10, 2019
fmassa
fmassa approved these changes Jun 12, 2019
View reviewed changes
Copy link
Contributor

@fmassa fmassa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@fmassa fmassa merged commit 7ee52a4 into facebookresearch:master Jun 12, 2019
@Johnqczhang
Copy link
Contributor

This change isn't backward-compatible when testing a pre-trained model specified in the .yaml file as the only argument --config-file passing to test_net.py, which makes the checkpointer fail to load the model file (despite it really exists) and initialize the model from scratch.

Adding a simple line of code can solve this problem, which is given in #894.

@Johnqczhang Johnqczhang mentioned this pull request Jun 14, 2019
Merged
Lyears pushed a commit to Lyears/maskrcnn-benchmark that referenced this pull request Jun 28, 2020
@wangg12 @fmassa
allow test for any checkpoint (facebookresearch#874)
50660cc
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@fmassa fmassa fmassa approved these changes

Assignees
No one assigned
Labels
CLA Signed Do not delete this pull request or issue due to inactivity.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@wangg12 @Johnqczhang @fmassa @facebook-github-bot