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

All varImp is 0 #18

Open
ChungsooKim opened this issue Oct 18, 2021 · 3 comments
Open

All varImp is 0 #18

ChungsooKim opened this issue Oct 18, 2021 · 3 comments

Comments

@ChungsooKim
Copy link
Collaborator

ChungsooKim commented Oct 18, 2021

As @jreps mentioned prior meeting (21-10-04),

I also found that all variable importance of the fitted model is zero.

image

However, I also figured out the final model from the fitDeepNNTorch function still had values of predictors as weights and biases in each layer. so we need to discuss how to deal with these values for model reproducibility

model$parameters()

image

image

@ChungsooKim ChungsooKim changed the title z All varImp is 0 Oct 18, 2021
@ChungsooKim
Copy link
Collaborator Author

we can input model agnostic importances (interpretability) metrics like SHAP, LIME, or permutation importance in the VarImp column. but torch in R doesn't seem to have tools calculating those metrics yet.

@chandryou
Copy link
Collaborator

I thought about SHAP, but it would take tons of time to screen all covariates...

@ChungsooKim
Copy link
Collaborator Author

Yes, that's right. we will use millions of covariates from millions of patients.
Then we can store all the original values of the weights for each layer in a separate column.
However, if we prepare the code for calculating the importance metric in advance, users can only calculate the metric if they want (optionally). I would try to get agnostic codes for the torch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants