PR #26059 Remove unnecessary `view`s of `position_ids`

Remove unnecessary `view`s of `position_ids` #26059

ArthurZucker merged 2 commits into huggingface:main from ramiro050:llama-remove-view

ArthurZucker approved these changes on 2023-09-08

ArthurZucker requested a review from

gante 2 years ago

ArthurZucker requested a review from

fxmarty 2 years ago

fxmarty approved these changes on 2023-09-15

fxmarty commented on 2023-09-15

ramiro050 requested a review from

ArthurZucker 2 years ago

ramiro050 requested a review from

fxmarty 2 years ago

ramiro050 changed the title ~~Remove unnecessary `view` of `position_ids` in `modeling_llama`~~ Remove unnecessary `view`s of `position_ids` 2 years ago

Remove unnecessary `view` of `position_ids` in `modeling_llama`

27e69a2d

ramiro050 force pushed 2 years ago

Remove no-op `view` of `position_ids` in rest of transformer models

a0887a6e

ramiro050 force pushed to a0887a6e 2 years ago

ArthurZucker commented on 2023-09-28

fxmarty approved these changes on 2023-10-06

ArthurZucker approved these changes on 2023-10-06

ArthurZucker merged 8878eb1b into main 2 years ago

ramiro050 deleted the llama-remove-view branch 2 years ago

Reviewers

ArthurZucker

fxmarty

gante

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

transformers Remove unnecessary `view`s of `position_ids` #26059 Merged

Remove unnecessary `view`s of `position_ids` #26059

transformers
Remove unnecessary `view`s of `position_ids`
#26059

Merged