Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Classification metrics overhaul: accuracy metrics (2/n) #4838
Classification metrics overhaul: accuracy metrics (2/n) #4838
Changes from 128 commits
6959ea0
0679015
35627b5
55fdaaf
35f8320
5cbf56a
9c33d0b
6562205
b97aef2
cbbc769
f45fc81
a04a71e
eaac5d7
c1108f0
277769b
4849298
22906a4
1034a71
02bd636
f97145b
536feaf
4241d7c
99d3c81
86d6c4d
54c98a0
bb11677
bdc4111
cde3997
05a54da
9a43a5e
3f4ad3c
a654e6a
7b2ef2b
16ab8f7
558276f
ecffe18
a907ade
725c7dd
41ad0b7
ca13e76
ede2c7f
c6e4de4
201d0de
f08edbc
523bae3
db24fae
35e3eff
dd6f8ea
c28aadf
4bfc688
323285e
0cb0eac
28acf4c
8e7a85a
829155e
768879d
e4d88e2
eeded45
b49cfdc
15ef14d
3d8f584
1568970
a9fa730
44ad276
96d40c8
cca430a
b0bde16
627d99a
de3defb
f3c47f9
a7e91a9
b7ced6e
e91e564
798ec03
7217924
a7c143e
531ae33
2eba226
a66cf31
e93f83e
89b09f8
e715437
d5daec8
c820060
b576de0
dae341b
b2d2b71
9b2a399
0952df2
c7fe698
e2bc0ab
acbd1ca
8801f8a
c32b36e
0314c7d
152cadf
022d6a6
9efc963
d992f7d
a726060
93c5d02
0813055
6bf714b
d12f1d6
a55cb46
d75eec3
6b3b057
6f218d4
98cb5f4
72ca3ac
474fbd0
03cccc3
de0213e
0fbf93c
1b8af65
c2c17f0
3c4f200
065c848
82d550e
279e4b9
c4e9aa2
827a544
eb9cb3c
69123af
863885a
811dc00
f68acc0
d47b559
3bf3c3a
4412c9c
a9a4847
44135c7
908e60f
23a997e
b3e458d
92f5f83
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just wondering, should threshold be None too by default?? because it's not used when we provider pred_labels, and use 0.5 by default when we have pred_probs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, I guess this could be done - but in a way that if pred_probs are passed in, None would default to 0.5. otherwise this would be a very disturbing breaking change for people used to using accuracy without extra params
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, that's what meant here
0.5 by default when we have pred_probs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got it, will add this in the next PR.