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

Occlusion: No multiply attr_output_fn with output #117

Merged
merged 1 commit into from
Apr 12, 2022

Conversation

chr5tphr
Copy link
Owner

  • do not multiply the output of attr_output_fn with the model's output,
    as this is neither intuitive nor flexible
  • instead, only sum over the attr_output_fn output to compute the
    batch-wise scores
  • in share/example/feed_forward.py: specify an attr_output function instead
    of a one-hot tensor, which uses only the output of the true label
    index, thus retaining the previous behavior and demonstration how to
    use the output function
  • in docs/source/how-to/use-attributors.rst: use a lambda expression for
    the occlusion attributor to demonstrate how to use score only for a
    certain index with the new interface

- do not multiply the output of attr_output_fn with the model's output,
  as this is neither intuitive nor flexible
- instead, only sum over the attr_output_fn output to compute the
  batch-wise scores
- in share/example/feed_forward.py: specify an attr_output function instead
  of a one-hot tensor, which uses only the output of the true label
  index, thus retaining the previous behavior and demonstration how to
  use the output function
- in docs/source/how-to/use-attributors.rst: use a lambda expression for
  the occlusion attributor to demonstrate how to use score only for a
  certain index with the *new* interface
@chr5tphr chr5tphr merged commit 1950513 into master Apr 12, 2022
@chr5tphr chr5tphr deleted the occlusion-no-multiply branch April 12, 2022 15:35
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

Successfully merging this pull request may close these issues.

1 participant