transformers
Idefics: Fix information leak with cross attention gate in modeling
#26839
Merged

Idefics: Fix information leak with cross attention gate in modeling #26839

leot13
leot13 fix image_attention gate in idefics modeling
aaf79cc0
HuggingFaceDocBuilderDev
leot13 update comment
3d5c7f4c
leot13 cleaner gating
b13ce277
leot13 fix gate condition
9c5cad2a
leot13 create attention gate once
69b8ab94
leot13 update comment
6d5179b1
leot13 update doc of cross-attention forward
92e1f504
leot13 improve comment
17403d92
leot13 leot13 requested a review from ArthurZucker ArthurZucker 2 years ago
leot13 leot13 marked this pull request as ready for review 2 years ago
leot13 Merge branch 'main' into fix-idefics-image-attention
f6ac847b
ArthurZucker
ArthurZucker commented on 2023-10-23
leot13 bring back no_images
598deb12
leot13 pass cross_attention_gate similarly to no_images gate
699838cc
leot13 add information on gate shape
7a45089b
leot13 fix no_images placement
ac493193
leot13 make tests for gate
6cc352cf
leot13
leot13 commented on 2023-10-26
leot13 take off no_images logic
71d444da
leot13 leot13 requested a review from ArthurZucker ArthurZucker 2 years ago
leot13 Merge branch 'main' into fix-idefics-image-attention
f5b26b94
leot13 leot13 requested a review from amyeroberts amyeroberts 2 years ago
amyeroberts
amyeroberts commented on 2023-10-30
leot13 update test based on comments
9a000da9
leot13 leot13 requested a review from amyeroberts amyeroberts 2 years ago
amyeroberts
amyeroberts approved these changes on 2023-11-02
ydshieh
leot13 raise value error if cross_attention_gate is None
a7ff77a5
ydshieh
leot13 send cross_attention_gate to device
054f8422
leot13 Revert "send cross_attention_gate to device"
b45351d3
leot13 send cross_attention_gate to device
58822578
leot13
ydshieh
leot13 fix device in test + nit
0ed25e3c
leot13
ydshieh
ydshieh
leot13
ydshieh
leot13 fill hidden_states with zeros instead of multiplying with the gate
0af2a820
leot13 style
69e7fe2b
leot13
ydshieh
ArthurZucker
ArthurZucker approved these changes on 2023-11-21
leot13 Update src/transformers/models/idefics/modeling_idefics.py
1840ccc7
leot13 Update src/transformers/models/idefics/modeling_idefics.py
54417d89
ArthurZucker ArthurZucker merged 851a4f70 into main 2 years ago

Login to write a write a comment.

Login via GitHub

Assignees
No one assigned
Labels
Milestone