Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Feature/auc_metrics #149

Merged
merged 39 commits into from
Jun 17, 2024
Merged

Feature/auc_metrics #149

merged 39 commits into from
Jun 17, 2024

Conversation

blondered
Copy link
Collaborator

@blondered blondered commented May 28, 2024 •
edited
Loading

@blondered blondered requested a review from feldlime May 28, 2024 19:31
@blondered blondered self-assigned this May 28, 2024
blondered
blondered commented May 28, 2024
View reviewed changes
rectools/metrics/auc.py Outdated Show resolved Hide resolved
tests/metrics/test_auc.py Outdated Show resolved Hide resolved
feldlime
feldlime reviewed Jun 1, 2024
View reviewed changes
.pylintrc Outdated Show resolved Hide resolved
rectools/metrics/base.py Outdated Show resolved Hide resolved
rectools/metrics/base.py Outdated Show resolved Hide resolved
rectools/metrics/base.py Outdated Show resolved Hide resolved
rectools/metrics/auc.py Outdated Show resolved Hide resolved
rectools/metrics/auc.py Outdated Show resolved Hide resolved
rectools/metrics/auc.py Outdated Show resolved Hide resolved
rectools/metrics/auc.py Outdated Show resolved Hide resolved
rectools/metrics/auc.py Outdated Show resolved Hide resolved
tests/metrics/test_auc.py Outdated Show resolved Hide resolved
feldlime
feldlime reviewed Jun 4, 2024
View reviewed changes
rectools/metrics/auc.py Outdated Show resolved Hide resolved
rectools/metrics/auc.py Outdated Show resolved Hide resolved
rectools/metrics/auc.py Outdated Show resolved Hide resolved
rectools/metrics/auc.py Outdated Show resolved Hide resolved
rectools/metrics/auc.py Outdated Show resolved Hide resolved
rectools/metrics/auc.py Outdated Show resolved Hide resolved
@codecov Codecov
Copy link

codecov bot commented Jun 14, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (9b3992e) to head (64e532f).
Report is 67 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##              main      #149    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files           45        54     +9     
  Lines         2242      3231   +989     
==========================================
+ Hits          2242      3231   +989

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

feldlime
feldlime reviewed Jun 15, 2024
View reviewed changes
rectools/metrics/auc.py Outdated Show resolved Hide resolved
rectools/metrics/auc.py Outdated Show resolved Hide resolved
@blondered blondered merged commit 0100f52 into main Jun 17, 2024
9 checks passed
@blondered blondered deleted the feature/pauc branch June 17, 2024 08:32
blondered added a commit that referenced this pull request Sep 23, 2024
@blondered
Feature/auc_metrics (#149)
1626ab4 
- Added PAP and PartialAUC ranking metrics
Closes #133, Closes #134
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@feldlime feldlime feldlime approved these changes

Assignees

@blondered blondered

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

pAp ranking metric pAUC ranking metric
2 participants
@blondered @feldlime