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.

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

[ENH] add PermutationForests and FeatureImportanceForests to sktree #125

Merged
merged 79 commits into from
Oct 5, 2023
Merged
Changes from 1 commit
Commits
Show all changes
79 commits
Select commit Hold shift + click to select a range
5a19fc6
ENH initialize with MIRF_AUC and MIRF_MV
PSSF23 Sep 11, 2023
f1a8e49
ENH add statistic alternatives
PSSF23 Sep 11, 2023
fd0b937
no axis=1 when taking posterior slice in MI
sampan501 Sep 12, 2023
efc2587
ENH add y-label permutation test to MIGHT
PSSF23 Sep 12, 2023
d1f7748
FIX rename import
PSSF23 Sep 12, 2023
d4abb4a
FIX, correct function param
PSSF23 Sep 12, 2023
a342482
FIX remove axis param & TST initialize test file
PSSF23 Sep 12, 2023
69c76a8
Adding modularity
adam2392 Sep 12, 2023
11f38c2
Merge branch 'might' of https://github.com/neurodata/scikit-tree into…
adam2392 Sep 12, 2023
b3ab11d
TST experiment with unit test
PSSF23 Sep 12, 2023
3ef1d7e
Merge branch 'main' into might
PSSF23 Sep 12, 2023
4ed31f8
FIX correct variable name
PSSF23 Sep 12, 2023
9114859
TST remove patch oblique tree tests
PSSF23 Sep 12, 2023
bbdfa33
Merged main
adam2392 Sep 12, 2023
5e1127f
Merge branch 'main' into might
adam2392 Sep 12, 2023
6b52608
WIP
adam2392 Sep 13, 2023
1a5ebe3
Linear model not working?
adam2392 Sep 14, 2023
d8da658
Adding permutation forest that seems to work
adam2392 Sep 15, 2023
3d666af
Upload notebook
adam2392 Sep 17, 2023
aab5df3
FIX correct MI calculation for MIGHT 2-class
PSSF23 Sep 18, 2023
972fe95
Correlated logit model
adam2392 Sep 18, 2023
f26825d
Patch up python code
adam2392 Sep 18, 2023
168935c
Add documentation
adam2392 Sep 18, 2023
01f2085
Adding posterior
adam2392 Sep 19, 2023
e9d07d8
Fix mesonb uild
adam2392 Sep 19, 2023
1dcd4f6
Fix unit-test
adam2392 Sep 19, 2023
37b1648
Fix docs errors
adam2392 Sep 19, 2023
5f9954d
Merge branch 'main' into might
adam2392 Sep 19, 2023
a28a842
Fix unit-test
adam2392 Sep 19, 2023
b88e12e
Clean up API
adam2392 Sep 19, 2023
579003f
Working clean code
adam2392 Sep 19, 2023
2fa68ce
Working clean code
adam2392 Sep 19, 2023
7fbdd38
Working clean code
adam2392 Sep 19, 2023
918f934
Working clean code
adam2392 Sep 19, 2023
bd02877
Fixed unit-tests
adam2392 Sep 19, 2023
1eb8604
Adding coverage
adam2392 Sep 19, 2023
8c1d15c
Updated example
adam2392 Sep 20, 2023
c40e866
Fix bug
adam2392 Sep 20, 2023
a6b1a0c
Fix example
adam2392 Sep 20, 2023
21f3b22
Fix unit-test
adam2392 Sep 20, 2023
c8af371
Remove unnecessary doc string
adam2392 Sep 20, 2023
fd5a63c
DOC correct result evaluation comment
PSSF23 Sep 20, 2023
aa404b0
Merge branch 'might' of https://github.com/neurodata/scikit-tree into…
adam2392 Sep 20, 2023
eee4cbc
Try redirect
adam2392 Sep 20, 2023
527397d
Try again
adam2392 Sep 20, 2023
fe848a4
Try again
adam2392 Sep 20, 2023
9a05235
Improve the checking inputs of feature importance forests
adam2392 Sep 21, 2023
dddacb1
Fix unit test
adam2392 Sep 21, 2023
33c9a74
Try again
adam2392 Sep 21, 2023
64f2017
Fix docs
adam2392 Sep 21, 2023
b8dc3a2
Fix pvalue sampling
adam2392 Sep 28, 2023
028f17d
Fixed pvalue issue
adam2392 Sep 28, 2023
2f06e76
Cleanup
adam2392 Sep 28, 2023
dc6dd29
Fix docs biuld
adam2392 Sep 28, 2023
dbf079e
Fix unit-test
adam2392 Sep 28, 2023
1c0f66b
Add update reshapes
adam2392 Sep 29, 2023
39aef2a
Add some todos and fixes from quick call w/ sambit/hao
adam2392 Oct 2, 2023
48d889b
set covariate_index to None by default and change self.n_classes
sampan501 Oct 3, 2023
bbb5c7c
Fix a few issues and consolidate todos
adam2392 Oct 3, 2023
563be04
Merge branch 'might' of https://github.com/neurodata/scikit-tree into…
adam2392 Oct 3, 2023
5ad1b44
Fix
adam2392 Oct 3, 2023
80ada68
Add clone to get estimators
adam2392 Oct 3, 2023
ff37740
ENH mark all default tests as MI and correct posterior return parameter
PSSF23 Oct 3, 2023
aed9179
FIX unify all variable names so posteriors are not saved twice
PSSF23 Oct 3, 2023
c716440
Add additional testing
adam2392 Oct 3, 2023
f6cb04b
Fix CI
adam2392 Oct 3, 2023
8df008d
Adding parallelization test
adam2392 Oct 3, 2023
7964d99
FIX remove extra print statememts
PSSF23 Oct 4, 2023
8b5a7d1
Remove numpy nanmean warnings and also bug fix of some code (#133)
adam2392 Oct 4, 2023
3a2279a
Add fixes
adam2392 Oct 4, 2023
37e6643
Merge branch 'might' of https://github.com/neurodata/scikit-tree into…
adam2392 Oct 4, 2023
3a4a4b4
Add parallelization to the tree building and predicting posteriors
adam2392 Oct 4, 2023
e91060f
ENH add MIGHT example notebook on AUC
PSSF23 Oct 4, 2023
efbd440
Consolidate parallleization
adam2392 Oct 4, 2023
8718b0f
Merge branch 'might' of https://github.com/neurodata/scikit-tree into…
adam2392 Oct 4, 2023
be16e5a
set default for covariate_index in ForestHT test
sampan501 Oct 5, 2023
26b5b5f
Add unit-test for small sample sizes
adam2392 Oct 5, 2023
80a4304
Final commit
adam2392 Oct 5, 2023
60d9c85
Release v0.2
adam2392 Oct 5, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions sktree/stats/forestht.py
adam2392 marked this conversation as resolved.
Show resolved Hide resolved
adam2392 marked this conversation as resolved.
Show resolved Hide resolved
Original file line number Diff line number Diff line change
Expand Up @@ -152,7 +152,7 @@ def statistic(
X: ArrayLike,
y: ArrayLike,
covariate_index: ArrayLike = None,
metric="mse",
metric="mi",
return_posteriors: bool = False,
check_input: bool = True,
**metric_kwargs,
Expand Down Expand Up @@ -272,9 +272,9 @@ def test(
X,
y,
covariate_index: ArrayLike = None,
metric: str = "mse",
metric: str = "mi",
n_repeats: int = 1000,
return_posteriors: bool = False,
return_posteriors: bool = True,
**metric_kwargs,
):
"""Perform hypothesis test using Coleman method.
Expand Down Expand Up @@ -302,7 +302,7 @@ def test(
n_repeats : int, optional
Number of times to sample the null distribution, by default 1000.
return_posteriors : bool, optional
Whether or not to return the posteriors, by default False.
Whether or not to return the posteriors, by default True.
**metric_kwargs : dict, optional
Additional keyword arguments to pass to the metric function.

Expand All @@ -322,7 +322,7 @@ def test(
y,
covariate_index=None,
metric=metric,
return_posteriors=True,
return_posteriors=return_posteriors,
check_input=False,
**metric_kwargs,
)
Expand All @@ -337,7 +337,7 @@ def test(
y,
covariate_index=covariate_index,
metric=metric,
return_posteriors=True,
return_posteriors=return_posteriors,
check_input=False,
**metric_kwargs,
)
Expand Down
Loading