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

Warn users when eval is skipped #624

Closed
wants to merge 1 commit into from

Conversation

gadcam
Copy link
Contributor

@gadcam gadcam commented Aug 16, 2018

Fix for #620 & #293.

@gadcam
Copy link
Contributor Author

gadcam commented Aug 29, 2018

@rbgirshick @ir413 Why not merging this one ?
This will avoid a lot of confusing situation where the detection is good and the AP is displayed as -1.
And at the same time, it does not change the computations' behavior.

Copy link

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

rbgirshick has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@rbgirshick
Copy link
Contributor

Hi @gadcam, can you format the warning messages so that the line length is <= 88 characters (80 chars + 10% slack)? Thanks.

@gadcam
Copy link
Contributor Author

gadcam commented Aug 30, 2018

Hi @rbgirshick
Now it should look better :) 80 characters not more, not less.

Copy link

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

rbgirshick has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

xzhewei pushed a commit to xzhewei/Detectron that referenced this pull request Jan 10, 2019
Summary:
Fix for facebookresearch#620 & facebookresearch#293.
Pull Request resolved: facebookresearch#624

Reviewed By: ir413

Differential Revision: D9574052

Pulled By: rbgirshick

fbshipit-source-id: f44cea71e5cf6e613d319322db62a60ecb814a75
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants