Commit 26cad25
Insights Fixes (#526)
Summary:
Insights example no longer worked after building master. This seems to be because of 2 issues:
1. Paths in setup.py needed to be updated after this refactoring #419 .
2. cifar_torchvision.pt seems to have been corrupted when this file was moved in #419, based on comparing the sizes in these 2 revisions:
https://github.com/pytorch/captum/blob/9bcf257f9b3811230f370ced1b0801d8df5aa9ee/captum/insights/models/cifar_torchvision.pt - > 244 KB
https://github.com/pytorch/captum/blob/master/captum/insights/attr_vis/models/cifar_torchvision.pt - >239 KB (master)
It seems like this was caused by the new file not being detected as binary, causing some corruption when pushed. This updates gitattributes to treat all .pt files as binary.
Pull Request resolved: #526
Reviewed By: bilalsal
Differential Revision: D24952285
Pulled By: vivekmig
fbshipit-source-id: 77b54caad7a4f2d2888ea59260603e00d62748ec1 parent 83a7e3e commit 26cad25
File tree
3 files changed
+4
-3
lines changed- captum/insights/attr_vis/models
3 files changed
+4
-3
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
| 2 | + | |
Binary file not shown.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
54 | 54 | | |
55 | 55 | | |
56 | 56 | | |
57 | | - | |
58 | | - | |
59 | | - | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
60 | 60 | | |
61 | 61 | | |
62 | 62 | | |
| |||
0 commit comments