Skip to content

Only show true/false/positive stats with validation data#507

Merged
danvk merged 2 commits intomasterfrom
novalid
Mar 3, 2015
Merged

Only show true/false/positive stats with validation data#507
danvk merged 2 commits intomasterfrom
novalid

Conversation

@danvk
Copy link
Contributor

@danvk danvk commented Mar 3, 2015

Fixes #376

Apparently this used to work but hasn't in some time due to a triple-equal null check on a non-existent property.

Review on Reviewable

@ihodes
Copy link
Member

ihodes commented Mar 3, 2015

Add a dpxdt test for the case where there is a validation set? LGTM otherwise!

@danvk
Copy link
Contributor Author

danvk commented Mar 3, 2015

There are some tests with validation sets (e.g. validated-run.png), but they don't show up here because not a single pixel changed.

danvk added a commit that referenced this pull request Mar 3, 2015
Only show true/false/positive stats with validation data
@danvk danvk merged commit 6d4e9bc into master Mar 3, 2015
@danvk danvk deleted the novalid branch March 19, 2015 22:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Don't show true/false/positive/negative when there's no validation data

2 participants