diffusers
Apply Occam's Razor in position embedding calculation
#11562
Merged
Overview
Commits
5
Changes
View On GitHub

Apply Occam's Razor in position embedding calculation #11562

yiyixuxu merged 5 commits into huggingface:main from tolgacangoz:remove-redundant-indexing
tolgacangoz
tolgacangoz36 days ago (edited 24 days ago)😄 2

Eliminate unnecessary indexing in the calculation of frequency values for the position embedding, simplifying the code.
Context: #11518 (comment)

@a-r-r-o-w @yiyixuxu

tolgacangoz fix: remove redundant indexing
9d3dc54a
tolgacangoz style
5ed995a1
tolgacangoz Merge branch 'main' into remove-redundant-indexing
d6bbc24d
tolgacangoz tolgacangoz changed the title Remove redundant indexing in position embedding calculation Apply Occam's Razor in position embedding calculation 36 days ago
a-r-r-o-w
a-r-r-o-w approved these changes on 2025-05-15
a-r-r-o-w36 days ago

Thanks!

a-r-r-o-w a-r-r-o-w requested a review from yiyixuxu yiyixuxu 36 days ago
HuggingFaceDocBuilderDev
HuggingFaceDocBuilderDev36 days ago

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

tolgacangoz Merge branch 'main' into remove-redundant-indexing
12051d58
tolgacangoz Merge branch 'main' into remove-redundant-indexing
4288f3ad
tolgacangoz tolgacangoz requested a review from a-r-r-o-w a-r-r-o-w 9 days ago
yiyixuxu
yiyixuxu approved these changes on 2025-06-11
yiyixuxu8 days ago

thanks! sorry for the delay

yiyixuxu yiyixuxu merged 47ef7946 into main 8 days ago
tolgacangoz
tolgacangoz8 days ago

Thanks for merging!

tolgacangoz tolgacangoz deleted the remove-redundant-indexing branch 8 days ago

Login to write a write a comment.

Login via GitHub

Reviewers
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Terms Of Use Privacy Policy Refund Policy Impressum