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

Signif subset #73

Merged
merged 83 commits into from
Jun 19, 2019
Merged

Signif subset #73

merged 83 commits into from
Jun 19, 2019

Conversation

saleiro
Copy link
Member

@saleiro saleiro commented Jun 18, 2019

No description provided.

lorenh516 and others added 30 commits November 30, 2018 17:04
@lorenh516
binary tp, fp, tn, fn cols for statistical significance
956a885
@lorenh516
IP: statistical significance notebook testing binary tp, fp, tn, fn c…
d892113 
…ols method in style of get_crosstabs for easy integration
@lorenh516
Calculate significance of differences between ref group and all other…
3114254 
… groups for label, score, tp, tn, fp, fn
@lorenh516
Merge branch 'master' of https://github.com/dssg/aequitas into issue_…
6f24add 
…51_significance
@lorenh516
mask and move significance methods to bias class
a34a01d
@lorenh516
ID ref groups for significance methods
3cfcb13
@lorenh516
Asignificance calculation, predefinied ref group bias method
b74480d
@lorenh516
Add significance calculation to all bias methods
bc3ea28
@lorenh516
IP: significance indicator in plotting disparity method
5dd1f45
@lorenh516
unmasked statistical significance signaling in treemaps (masked still…
65bacce 
… has error)
@lorenh516
masked statistical significance signaling in treemaps complete for al…
abb30c3 
…l disparity metrics
@lorenh516
Add method descriptions, dynamic significance asterisks in treemaps
752d179
@lorenh516
rm test statistical siginificance
c521951
@lorenh516
Layer check for siginificance columns in df before attempting treemap…
9166285 
… significance asterisks.
@lorenh516
remove hard-coded metrics to check from significance from majority bi…
6783610 
…as calculation
@lorenh516
Delete encodings.xml
d2c65c3
@lorenh516
progress on passing tests with min_metric designation
89105d2
@lorenh516
begin min_metric bias attempt w/o multiple groupbys
d2e7cd1
@lorenh516
threshold cutoff exploration
2a80a80
@lorenh516
NaN error fixed on min_metric testing. To do: verify numpy version re…
40a5869 
…: TypeError on numpy boolean subtract
@lorenh516
min_metric now possible with test 3 and test 4 csv
406e3ea
@lorenh516
fix for type error on numpy boolean subtract
969577d
@lorenh516
clean run of ipynb
940fb34
@lorenh516
ensure Plot() can handle multiple models
68d995f
@lorenh516
ensure Plot() can handle multiple models
8ed9712
@jesteria @lorenh516
Update src/aequitas/bias.py
aad208b 
value error if non-existing column provided for significance calcs

Co-Authored-By: lorenh516 <loren.hinkson@gmail.com>
@lorenh516
review comments
459f302
@lorenh516
Merge remote-tracking branch 'origin/issue_51_significance' into issu…
b446b23 
…e_51_significance
@lorenh516
breaking up long lines so easier to read
072dab9
@lorenh516
fix conflict
2daa2e0
lorenh516 and others added 28 commits May 21, 2019 18:29
@saleiro saleiro merged commit a61ef33 into master Jun 19, 2019
@sgpjesus sgpjesus deleted the signif_subset branch February 12, 2024 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@saleiro @lorenh516 @jesteria