-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Classification metrics overhaul: precision & recall (4/n) #4842
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
Merged
rohitgr7
merged 92 commits into
Lightning-AI:release/1.2-dev
from
tadejsv:cls_metrics_precision_recall
Jan 18, 2021
Merged
Changes from all commits
Commits
Show all changes
92 commits
Select commit
Hold shift + click to select a range
6959ea0
Add stuff
tadejsv 0679015
Change metrics documentation layout
tadejsv 35627b5
Add stuff
tadejsv 0282f3c
Add stat scores
tadejsv 55fdaaf
Change testing utils
tadejsv 35f8320
Merge branch 'cls_metrics_input_formatting' into cls_metrics_accuracy
tadejsv dd05912
Merge branch 'cls_metrics_input_formatting' into cls_metrics_stat_scores
tadejsv 5cbf56a
Replace len(*.shape) with *.ndim
tadejsv 9c33d0b
More descriptive error message for input formatting
tadejsv 6562205
Replace movedim with permute
tadejsv b97aef2
Merge branch 'cls_metrics_input_formatting' into cls_metrics_accuracy
tadejsv 74261f7
Merge branch 'cls_metrics_input_formatting' into cls_metrics_stat_scores
tadejsv cbbc769
PEP 8 compliance
tadejsv 33166c5
WIP
tadejsv 801abe8
Add reduce_scores function
tadejsv fb181ed
Merge branch 'cls_metrics_accuracy' into cls_metrics_stat_scores
tadejsv fbebd34
Temporarily add back legacy class_reduce
tadejsv b3d1b8b
Merge branch 'cls_metrics_stat_scores' into cls_metrics_precision_recall
tadejsv f45fc81
Division with float
tadejsv 3fdef40
Merge branch 'cls_metrics_accuracy' into cls_metrics_stat_scores
tadejsv 452df32
Merge branch 'cls_metrics_stat_scores' into cls_metrics_precision_recall
tadejsv 9d44a26
PEP 8 compliance
tadejsv 82c3460
Merge branch 'cls_metrics_stat_scores' into cls_metrics_precision_recall
tadejsv 5ce7cd9
Remove precision recall
tadejsv 3b70270
Replace movedim with permute
tadejsv f1ae7b2
Add back tests
tadejsv 04a5066
Add empty newlines
tadejsv 2033063
Add precision recall back
tadejsv 9dc7bea
Add empty line
tadejsv 6ba5566
Merge branch 'cls_metrics_stat_scores' into cls_metrics_precision_recall
tadejsv a9640f6
Fix permute
tadejsv 9bc0f4c
Merge branch 'cls_metrics_stat_scores' into cls_metrics_precision_recall
tadejsv 692392c
Fix some issues with old versions of PyTorch
tadejsv a04a71e
Style changes in error messages
tadejsv eaac5d7
More error message style improvements
tadejsv c1108f0
Fix typo in docs
tadejsv 277769b
Add more descriptive variable names in utils
tadejsv 4849298
Change internal var names
tadejsv 22906a4
Merge remote-tracking branch 'upstream/master' into cls_metrics_input…
tadejsv 1034a71
Merge branch 'cls_metrics_input_formatting' into cls_metrics_accuracy
tadejsv ebcdbeb
Merge branch 'cls_metrics_accuracy' into cls_metrics_stat_scores
tadejsv 8040d02
Merge branch 'cls_metrics_stat_scores' into cls_metrics_precision_recall
tadejsv 96a1a70
Merge branch 'master' into cls_metrics_precision_recall
tchaton 500d22f
Merge remote-tracking branch 'upstream/release/1.2-dev' into cls_metr…
tadejsv cac0b85
Revert unwanted changes
tadejsv d043384
Revert unwanted changes pt 2
tadejsv d6bc69b
Update metrics interface
tadejsv d6559f2
Add top_k parameter
tadejsv 0b8a2fd
Add back reduce function
tadejsv 777f1de
Merge branch 'release/1.2-dev' into cls_metrics_precision_recall
Borda 0314a62
Add stuff
tadejsv be60aa1
Merge branch 'cls_metrics_precision_recall' of github.com:tadejsv/pyt…
tadejsv a9b0b93
Merge remote-tracking branch 'upstream/release/1.2-dev' into cls_metr…
tadejsv e96a442
PEP3
tadejsv b6de27a
Add depreciation
tadejsv 24adfe8
PEP8
tadejsv 660d4b1
Deprecate param
tadejsv 6b018d9
PEP8
tadejsv 9fdfcf6
Fix and simplify testing for older PT versions
tadejsv 0ad7368
Merge remote-tracking branch 'upstream/release/1.2-dev' into cls_metr…
tadejsv 88fd8cc
Update Changelog
tadejsv 6a7b86f
Remove redundant import
tadejsv df6365a
Add tests to increase coverage
tadejsv 17c680d
Merge remote-tracking branch 'upstream/release/1.2-dev' into cls_metr…
tadejsv 5e0dfbd
Remove zero_division
tadejsv 69b3305
Merge remote-tracking branch 'upstream/release/1.2-dev' into cls_metr…
tadejsv 5658ee5
fix zero_division
tadejsv 6ab9002
Add zero_div + edge case tests
tadejsv 571cdd8
Reorder cls metric args
tadejsv fff6b8b
Add back quotes for is_multiclass
tadejsv 46d7363
Add precision_recall and tests
tadejsv b6e375d
PEP8
tadejsv 0ef081b
Fix docs
tadejsv 3d0c985
Fix docs
tadejsv 92f7c5f
Merge branch 'release/1.2-dev' into cls_metrics_precision_recall
tadejsv e69a71a
Update
tadejsv f5e3676
Merge branch 'cls_metrics_precision_recall' of github.com:tadejsv/pyt…
tadejsv b6f6576
Merge remote-tracking branch 'upstream/release/1.2-dev' into cls_metr…
tadejsv b2bd166
Change precision_recall output
tadejsv 5eac1f4
PEP8/isort
tadejsv de4fb12
Add method _get_final_stats
tadejsv 3142557
Fix depr test
tadejsv 5b85787
Merge branch 'release/1.2-dev' into cls_metrics_precision_recall
tadejsv e517456
Merge branch 'release/1.2-dev' into cls_metrics_precision_recall
tadejsv 371f5bb
Add comment to deprecation tests
tadejsv b787661
Merge branch 'cls_metrics_precision_recall' of github.com:tadejsv/pyt…
tadejsv 937ae12
isort
tadejsv ee6fea5
Merge branch 'release/1.2-dev' into cls_metrics_precision_recall
Borda 728980e
Apply suggestions from code review
tadejsv 7a2bf23
Merge remote-tracking branch 'upstream/release/1.2-dev' into cls_metr…
tadejsv 7511443
Add typing to test
tadejsv 67d9702
Add matc str to pytest.raises
tadejsv File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.