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

Loss ratio senstir #43

Open
wants to merge 110 commits into
base: main
Choose a base branch
from
Open

Loss ratio senstir #43

wants to merge 110 commits into from

Conversation

aldopareja
Copy link
Contributor

fixes #38

Your Name and others added 30 commits July 14, 2022 19:39
added wasserstein distance, it's respective tests and modified the re…
e86bcd6 
…quirements
modified setup.py to avoid clashes
6171568
Revert "modified setup.py to avoid clashes"
05181a4 
This reverts commit 6171568.
@aldopareja
added the distance to documentation
23335cc
@aldopareja
iterim commit, fixing
b2e1b8b
@aldopareja
fixed test_common distances
5650faa
@aldopareja
problems in merge
30e5591
@aldopareja
fixed merge from main
1488423
@aldopareja
made the batched version a parameter of the forward method and modifi…
67cd46b 
…ed the test accordingly
@aldopareja
merging batched mahalanobis
700db6c
@aldopareja
removed unused code
2630877
@aldopareja
Merge branch 'batched_mahalanobis_distance-2' into wasserstein-distan…
977b108 
…ce-ranking-2
@aldopareja
made the wasserstein distance into a new class inheriting from distan…
2289025 
…ce, only supporting mahalanobis distance parameters at the moment. Also modified tests and documentation accordingly
@aldopareja
iterim commit, need to bring some changes from another branch
4cb3eeb
@aldopareja
wrote the auditor code, will work on test and documentation now
5077064
@aldopareja
iterim commit, need to modify the batched pairwise mahalanobis distance
15295ed
@aldopareja
added better documentation to wasserstein distance
e08c5b7
@aldopareja
Merge branch 'wasserstein-distance-ranking-2' into ranking-auditor
077ec5e
@aldopareja
fixed the input_distance and output distance size differences
961be1f
@aldopareja
finished code for sestir auditor and its test, also used black on them
0f3c775
@aldopareja
removed all changes from this branch since I ended up not using it
ffa9c8b
@aldopareja
removed batched mahalanobis
9d27b9c
@aldopareja
Merge branch 'wasserstein-distance-ranking-2' into ranking-auditor
05c2440
@aldopareja
removing batched mahalanobis
8416928
@aldopareja
removing batched mahalanobis
1aae6a9
@aldopareja
removing batched mahalanobis
b9b0cb3
@aldopareja
modified the test, it was too stringent
4c68865
@aldopareja
fixed merge coflict
458fc33
@aldopareja
removed stochasticity from the test
2ffc393
@aldopareja
removed stochasticity from the test
5b7d5bf
aldopareja added 18 commits August 4, 2022 23:49
@aldopareja
plackett-luce requires torch 1.12 so we will have to go with it
c6361d9
@aldopareja
plackett-luce requires torch 1.12 so we will have to go with it
cd93fb6
@aldopareja
plackett-luce requires torch 1.12 so we will have to go with it
1a3b272
@aldopareja
plackett-luce requires torch 1.12 so we will have to go with it
29719df
@aldopareja
using POT instead of geomloss now
e64d4c1
@aldopareja
Merge branch 'wasserstein-distance-ranking-2' into ranking-auditor
8b388fe
@aldopareja
fixed conflict
750e7d5
@aldopareja
fixed conflict
769ae7c
@aldopareja
fixed conflict
75c2c79
@aldopareja
fixed conflict
fea23bc
@aldopareja
plackett-luce requires torch 1.12 so we will have to go with it
e3d2829
@aldopareja
Merge branch 'plackett_luce-2' into senstir-algo
e618595
@aldopareja
Merge branch 'senstir-algo' into loss_ratio_senstir
75347de
@aldopareja
we need latest functorch to work with pytorch 1.12
2140bab
@aldopareja
we need latest functorch to work with pytorch 1.12
69a3432
@aldopareja
updated to torch 1.12.1 since they have fixed the bug
4749526
@aldopareja
updated to torch 1.12.1 since they have fixed the bug
124e857
@aldopareja
updated to torch 1.12.1 since they have fixed the bug
76b82d4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MayankAgarwal MayankAgarwal Awaiting requested review from MayankAgarwal

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

add ranking auditor capability to use loss ratio
1 participant
@aldopareja