More sklearn-compatible algorithms #318
Merged
Conversation
This file contains 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
Collaborator
hoffmansc
commented
Jun 28, 2022
hoffmansc
commented
- RejectOptionClassification (Port post-processing algorithms to sklearn-compatible API #151)
- LearnedFairRepresentations (Port pre-processing algorithms to sklearn-compatible API #154)
- option to suppress zero division warnings in metrics
- consolidate dataset loading in conftest.py
* dataset loading is more similar to sklearn.datasets * label binarization is now done outside standardize_dataset * metrics use 'groups' and 'priv_group' to signify priv/unpriv split
removed Reweighing.sample_weight_ attribute
Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
* postprocessors need 'requires_proba' tag * no default postprocessor * fix prefit, score PostProcessingMeta bugs Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
* add fit_predict * add sample_weight warning Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
|
This pull request introduces 1 alert and fixes 1 when merging f978b5f into 89f700b - view on LGTM.com new alerts:
fixed alerts:
|
mnagired
approved these changes
Jun 29, 2022
Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
|
This pull request introduces 1 alert and fixes 1 when merging 35f87a7 into 38368c5 - view on LGTM.com new alerts:
fixed alerts:
|
Illia-Kryvoviaz
pushed a commit
to Illia-Kryvoviaz/AIF360
that referenced
this pull request
Jun 7, 2023
* new ROC and LFR implementations * allow prot_attr/target input to be Series * LFR uses pytorch to calculate grad * change defaults in metaestimators * clean up tests * infer proba behavior for PostProcessingMeta from estimator 'requires_proba' tag * fix prefit, score PostProcessingMeta bugs * propagate classes_ in metaestimators Signed-off-by: Samuel Hoffman <hoffman.sc@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.