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

WIP sklearn compatible ThresholdOptimizer #282

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

WIP sklearn compatible ThresholdOptimizer #282

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
06da6b3
sklearn compatible ThresholdOptimizer
adrinjalali Feb 2, 2020
10754c4
remove _postprosessing.py
adrinjalali Feb 2, 2020
78b8963
apply suggestions
adrinjalali Feb 5, 2020
6c0cc01
add warm_start
adrinjalali Feb 6, 2020
390544e
further fixes, passing a part of the tests
adrinjalali Feb 6, 2020
f7326a6
further fixes, passing a part of the tests
adrinjalali Feb 6, 2020
df89f19
accept arbitrary input type for sensitive features
adrinjalali Feb 6, 2020
fa2c809
use log loss to have predict_proba
adrinjalali Feb 6, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
18 changes: 15 additions & 3 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,10 +1,22 @@
# simple makefile to simplify repetitive build env management tasks

PYTHON ?= python
PYTEST ?= pytest

all: clean inplace test-unit

clean:
git clean -xfd

in: inplace # just a shortcut
inplace:
$(PYTHON) setup.py build_ext -i

test-coverage:
python -m pytest test -m "not notebooks" --ignore=test/perf --ignore=test/install --cov=fairlearn --cov-report=xml --cov-report=html
$(PYTEST) test -m "not notebooks" --ignore=test/perf --ignore=test/install --cov=fairlearn --cov-report=xml --cov-report=html

test-unit:
python -m pytest ./test/unit
$(PYTEST) ./test/unit

test-perf:
python -m pytest ./test/perf
$(PYTEST) ./test/perf
2 changes: 0 additions & 2 deletions fairlearn/postprocessing/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,8 @@
learn how to adjust the predictor's output from the training data.
"""

from ._postprocessing import PostProcessing # noqa: F401
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this probably warrants a description in the CHANGES.md as well :-)

from ._threshold_optimizer import ThresholdOptimizer # noqa: F401

__all__ = [
"PostProcessing",
"ThresholdOptimizer"
]
108 changes: 0 additions & 108 deletions fairlearn/postprocessing/_postprocessing.py
View file
Open in desktop

This file was deleted.