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

Add more algorithms #8

Merged
merged 14 commits into from
Dec 28, 2022
Merged

Add more algorithms #8

merged 14 commits into from
Dec 28, 2022

Conversation

adamwawrzynski
Copy link
Collaborator

@adamwawrzynski adamwawrzynski commented Dec 22, 2022 •
edited
Loading

Description

Add more algorithms from captum package.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

  • manual UI testing

Checklist:

  • I have updated all related pyproject.toml files.
  • I have updated lock files.
  • I have set an Assignee and Reviewers.
  • I have labeled the PR correctly.
  • I have performed a self-review of my own code.
  • I have covered my code and changes with unit tests.
  • I have annotated types as extensively as possible.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • All code references and dependencies will work.

Adam Wawrzyński added 8 commits December 22, 2022 16:19
Change kwargs argument name, unify 'layer' argument name passed to ex…
f3fdc1d 
…plainers

Signed-off-by: Adam Wawrzyński <adam.wawrzynski@reasonfieldlab.com>
Fix docstrings of create_explainer function
a2d09e0 
Signed-off-by: Adam Wawrzyński <adam.wawrzynski@reasonfieldlab.com>
Add Saliency algorithm
2907bde 
Signed-off-by: Adam Wawrzyński <adam.wawrzynski@reasonfieldlab.com>
Add DeepLIFT algorithm
bc6c616 
Signed-off-by: Adam Wawrzyński <adam.wawrzynski@reasonfieldlab.com>
Add DeepLIFT SHAP algorithm
9c2fca2 
Signed-off-by: Adam Wawrzyński <adam.wawrzynski@reasonfieldlab.com>
Add Deconvolution algorithm
526fe50 
Signed-off-by: Adam Wawrzyński <adam.wawrzynski@reasonfieldlab.com>
Add Input x Gradient algorithm
348e323 
Signed-off-by: Adam Wawrzyński <adam.wawrzynski@reasonfieldlab.com>
Add Layer Conductance algorithm
f129175 
Signed-off-by: Adam Wawrzyński <adam.wawrzynski@reasonfieldlab.com>
@adamwawrzynski adamwawrzynski self-assigned this Dec 22, 2022
Add standardization of matrices, generate figure with respect to pres…
0c7609a 
…ence of negative attributions

Signed-off-by: Adam Wawrzyński <adam.wawrzynski@reasonfieldlab.com>
@adamwawrzynski adamwawrzynski added enhancement New feature or request bug Something isn't working labels Dec 23, 2022
@adamwawrzynski adamwawrzynski marked this pull request as ready for review December 23, 2022 11:09
kamilrzechowski
kamilrzechowski approved these changes Dec 27, 2022
View reviewed changes
src/explainer/base_explainer.py Outdated Show resolved Hide resolved
src/explainer/conductance.py Outdated Show resolved Hide resolved
Adam Wawrzyński added 2 commits December 28, 2022 15:16
Revert visualization error handling due to #12 PR
2232aa8 
Signed-off-by: Adam Wawrzyński <adam.wawrzynski@reasonfieldlab.com>
Mark model argument type with TODO
6404c06 
Signed-off-by: Adam Wawrzyński <adam.wawrzynski@reasonfieldlab.com>
@adamwawrzynski adamwawrzynski removed the bug Something isn't working label Dec 28, 2022
Adam Wawrzyński added 3 commits December 28, 2022 16:24
Merge branch 'main' into feat/captum_more_algorithms
2cdbb7f 
Signed-off-by: Adam Wawrzyński <adam.wawrzynski@reasonfieldlab.com>
Remove incorrect standardization method
56a15df 
Signed-off-by: Adam Wawrzyński <adam.wawrzynski@reasonfieldlab.com>
Bump patch version
3ca6ff9 
Signed-off-by: Adam Wawrzyński <adam.wawrzynski@reasonfieldlab.com>
@adamwawrzynski adamwawrzynski merged commit 4b54012 into main Dec 28, 2022
@rozek1997 rozek1997 deleted the feat/captum_more_algorithms branch January 16, 2023 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kamilrzechowski kamilrzechowski kamilrzechowski approved these changes

@adamjankaczmarek adamjankaczmarek Awaiting requested review from adamjankaczmarek

@rpytel1 rpytel1 Awaiting requested review from rpytel1

Assignees

@adamwawrzynski adamwawrzynski

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@adamwawrzynski @kamilrzechowski