Skip to content

Commit

Permalink
Add test coverage for futures in ShapleyValueSampling 1/n (#1491)
Browse files Browse the repository at this point in the history
Summary:

This diffs adds more testing coverage for attribute_future to ShapleyValueSampling unit tests that handle simple shapley sampling and multi shapley sampling using the BasicModel_MultiLayer model

note: decided to parameterize the high-level unit tests since I did not want to clutter the unit test file with tests which were doing the same calculations but with the pytorch futures api. Separated the model and shapley assert methods between non-future vs future so that the differences between the two would be clear

Reviewed By: cyrjano

Differential Revision: D68229981
  • Loading branch information
jjuncho authored and facebook-github-bot committed Jan 22, 2025
1 parent cf9062f commit 3789f8d
Showing 1 changed file with 251 additions and 123 deletions.
Loading

0 comments on commit 3789f8d

Please sign in to comment.