-
Notifications
You must be signed in to change notification settings - Fork 31
About all_text_embedding #27
Copy link
Copy link
Open
Description
Dear Author, while reviewing the code, I noticed the following issue:all_text_embedding = self.get_input_embeddings()(input_ids.clamp(min=0)).detach()This all_text_embedding includes embeddings for the answer (e.g., GPT output), which are then fed into prefusion_layers for fusion with other features. Is this design correct?
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels