-
-
Notifications
You must be signed in to change notification settings - Fork 17.1k
python312Packages.captum: init at 0.7.0 #356087
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
Conversation
0d692ca to
4c8a6cb
Compare
5926d12 to
8ad9b7a
Compare
|
@ofborg build python312Packages.captum |
|
1 similar comment
|
|
8ad9b7a to
fbadcd5
Compare
| # Failing tests: AssertionError: 1.0 != 0.5 within 0.0001 delta (0.5 difference) | ||
| "test_TCAV_0_0" | ||
| "test_TCAV_0_1" | ||
| "test_TCAV_0_1_attr_to_inputs" | ||
| "test_TCAV_1_0_1" | ||
| "test_TCAV_1_0_b" | ||
| "test_TCAV_1_1_a" | ||
| "test_TCAV_1_1_a_wo_acc_metric" | ||
| "test_TCAV_1_1_b" | ||
| "test_TCAV_1_1_c" | ||
| "test_TCAV_1_1_d" | ||
| "test_TCAV_x_1_0_1" | ||
| "test_TCAV_x_1_0_1_attr_to_inputs" | ||
| "test_TCAV_x_1_0_1_w_flipped_class_id" | ||
| "test_TCAV_x_1_0_a" | ||
| "test_TCAV_x_1_0_b" | ||
| "test_TCAV_x_1_1_a" | ||
| "test_TCAV_x_1_1_b" | ||
| "test_TCAV_x_1_1_c" | ||
| "test_TCAV_x_1_1_c_concept_order_changed" | ||
| "test_TCAV_x_1_1_" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe add tests/concept/test_tcav.py to disabledTestPaths instead.
Also, could you please report this upstream ?
fbadcd5 to
639d1c7
Compare
|
639d1c7 to
a2c826f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
| homepage = "https://github.com/pytorch/captum"; | ||
| license = lib.licenses.bsd3; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| homepage = "https://github.com/pytorch/captum"; | |
| license = lib.licenses.bsd3; | |
| homepage = "https://github.com/pytorch/captum"; | |
| changelog = "https://github.com/pytorch/captum/releases/tag/v${version}"; | |
| license = lib.licenses.bsd3; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Zut.... Je corrige ça asap.
Summary: While packaging this module for Nix (NixOS/nixpkgs#356087), I noticed during the tests that `flask` and `flask-compress` modules were missing from the `setup.py` file. Pull Request resolved: #1442 Reviewed By: cyrjano Differential Revision: D71814035 Pulled By: jjuncho fbshipit-source-id: 4e1cbc9e2e43dd88657063fade94405094bb4190
Things done
nix.conf? (See Nix manual)sandbox = relaxedsandbox = truenix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)Add a 👍 reaction to pull requests you find important.