Skip to content

Conversation

@aobo-y
Copy link
Contributor

@aobo-y aobo-y commented Dec 1, 2025

Summary: following the discussion in D87818563, MMImageMaskInput can be generalized for non-LLM image model. So we rename the class as ImageMaskInput and make the processor_fn as optional. By default, if processor_fn is not given, the model input will be the (ablated) image

Reviewed By: craymichael

Differential Revision: D87946132

Summary:

as title, as there is only one color used to mask out

Reviewed By: Ayush-Warikoo

Differential Revision: D87944294
Summary: following the discussion in D87818563, `MMImageMaskInput` can be generalized for non-LLM image model. So we rename the class as `ImageMaskInput` and make the `processor_fn` as optional. By default, if `processor_fn` is not given, the model input will be the (ablated) image

Reviewed By: craymichael

Differential Revision: D87946132
@meta-cla meta-cla bot added the cla signed label Dec 1, 2025
@meta-codesync
Copy link
Contributor

meta-codesync bot commented Dec 1, 2025

@aobo-y has exported this pull request. If you are a Meta employee, you can view the originating Diff in D87946132.

aobo-y added a commit to aobo-y/captum that referenced this pull request Dec 1, 2025
Summary:

following the discussion in D87818563, `MMImageMaskInput` can be generalized for non-LLM image model. So we rename the class as `ImageMaskInput` and make the `processor_fn` as optional. By default, if `processor_fn` is not given, the model input will be the (ablated) image

Reviewed By: craymichael

Differential Revision: D87946132
aobo-y added a commit to aobo-y/captum that referenced this pull request Dec 1, 2025
Summary:

following the discussion in D87818563, `MMImageMaskInput` can be generalized for non-LLM image model. So we rename the class as `ImageMaskInput` and make the `processor_fn` as optional. By default, if `processor_fn` is not given, the model input will be the (ablated) image

Reviewed By: craymichael

Differential Revision: D87946132
@meta-codesync meta-codesync bot closed this in f536453 Dec 2, 2025
@meta-codesync
Copy link
Contributor

meta-codesync bot commented Dec 2, 2025

This pull request has been merged in f536453.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants