Skip to content

GenerationAttack and HuggingFace predictor updates#84

Closed
mgrange1998 wants to merge 1 commit intofacebookresearch:mainfrom
mgrange1998:export-D87341640
Closed

GenerationAttack and HuggingFace predictor updates#84
mgrange1998 wants to merge 1 commit intofacebookresearch:mainfrom
mgrange1998:export-D87341640

Conversation

@mgrange1998
Copy link
Contributor

Summary:
This change updates GenerationAttack and HuggingFace predictor with the following functionality.

  • Fixes "batch_size=self.batch_size" s.t it is propagated downstream properly. Updated the default value to 1 to reflect intended behavior
  • Adds "_generate_decode_logic" helper to encapsulate calling "model.generate"
  • Extends generation functionality to not include prompt when decoding

Differential Revision: D87341640

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Nov 18, 2025
@meta-codesync
Copy link

meta-codesync bot commented Nov 18, 2025

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

mgrange1998 added a commit to mgrange1998/PrivacyGuard-1 that referenced this pull request Nov 21, 2025
Summary:

This change updates GenerationAttack and HuggingFace predictor with the following functionality. 

- Fixes "batch_size=self.batch_size" s.t it is propagated downstream properly. Updated the default value to 1 to reflect intended behavior
- Adds "_generate_decode_logic" helper to encapsulate calling "model.generate"
- Extends generation functionality to not include prompt when decoding

Differential Revision: D87341640
mgrange1998 added a commit to mgrange1998/PrivacyGuard-1 that referenced this pull request Nov 21, 2025
Summary:

This change updates GenerationAttack and HuggingFace predictor with the following functionality. 

- Fixes "batch_size=self.batch_size" s.t it is propagated downstream properly. Updated the default value to 1 to reflect intended behavior
- Adds "_generate_decode_logic" helper to encapsulate calling "model.generate"
- Extends generation functionality to not include prompt when decoding

Differential Revision: D87341640
Summary:

This change updates GenerationAttack and HuggingFace predictor with the following functionality. 

- Fixes "batch_size=self.batch_size" s.t it is propagated downstream properly. Updated the default value to 1 to reflect intended behavior
- Adds "_generate_decode_logic" helper to encapsulate calling "model.generate"
- Extends generation functionality to not include prompt when decoding

Reviewed By: lucamelis

Differential Revision: D87341640
mgrange1998 added a commit to mgrange1998/PrivacyGuard-1 that referenced this pull request Nov 24, 2025
Summary:

This change updates GenerationAttack and HuggingFace predictor with the following functionality. 

- Fixes "batch_size=self.batch_size" s.t it is propagated downstream properly. Updated the default value to 1 to reflect intended behavior
- Adds "_generate_decode_logic" helper to encapsulate calling "model.generate"
- Extends generation functionality to not include prompt when decoding

Reviewed By: lucamelis

Differential Revision: D87341640
@meta-codesync meta-codesync bot closed this in 0cf30f9 Nov 24, 2025
@meta-codesync
Copy link

meta-codesync bot commented Nov 24, 2025

This pull request has been merged in 0cf30f9.

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

Labels

CLA Signed This label is managed by the Meta Open Source bot. fb-exported Merged meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants