Skip to content

Conversation

@cyrjano
Copy link
Contributor

@cyrjano cyrjano commented Nov 12, 2025

Summary: Create Progress Protocol and NullProgress to simplify the progress bar.

Differential Revision: D86798999

@meta-cla meta-cla bot added the cla signed label Nov 12, 2025
@meta-codesync
Copy link
Contributor

meta-codesync bot commented Nov 12, 2025

@cyrjano has exported this pull request. If you are a Meta employee, you can view the originating Diff in D86798999.

…eta-pytorch#1662)

Summary:

Improve documentation of the _format method and cleanup a few typing errors.

Reviewed By: styusuf

Differential Revision: D86432880
…-pytorch#1663)

Summary:

This diff moves methods out of the `FeatureAblation` class and makes them free methods. The changes include creating a new method `_parse_forward_out` to force forward output type assertion and conversion, and modifying the `add_one_back` module to use the new methods. The `attr/fb/add_one_back.py` file has been modified to use the new methods. The `attr/fb/within_group_utils.py` file has also been modified to use the new methods. The `attr/fb/test_within_group_utils.py` file has been

Differential Revision: D86785624
Summary:

This diff contains minor refactoring changes to the FeatureAblation methods in the Captum library to improve readability.

Differential Revision: D86791984
Summary:

Create Progress Protocol and NullProgress to simplify the progress bar.

Differential Revision: D86798999
cyrjano added a commit to cyrjano/captum that referenced this pull request Nov 13, 2025
Summary:

Create Progress Protocol and NullProgress to simplify the progress bar.

Reviewed By: sarahtranfb

Differential Revision: D86798999
@meta-codesync meta-codesync bot closed this in 6fa02a2 Nov 13, 2025
@meta-codesync
Copy link
Contributor

meta-codesync bot commented Nov 13, 2025

This pull request has been merged in 6fa02a2.

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