Use parallel_for in elementwise_util - #9243

Merged
swolchok merged 162 commits into
mainfrom
gh/swolchok/358/head
Mar 18, 2025
Merged

Use parallel_for in elementwise_util#9243
swolchok merged 162 commits into
mainfrom
gh/swolchok/358/head

Conversation

@swolchok

@swolchokswolchok commented Mar 13, 2025

Copy link
Copy Markdown
Contributor

More straightforward rollout.

(Parallelizing over BroadcastIndexesRange is ugly, but so far we have exactly two instances of it. If it keeps cropping up, we can add a utility to make it nicer.)

swolchok added 30 commits March 4, 2025 11:35
@swolchok
Update
d0b11e8
[ghstack-poisoned]
@swolchok
Update
9437be1
[ghstack-poisoned]
@swolchok
Update
643e10e
[ghstack-poisoned]
@swolchok
Update
6f2842b
[ghstack-poisoned]
@swolchok
Update
e47dfeb
[ghstack-poisoned]
@swolchok
Update
231ebc3
[ghstack-poisoned]
@swolchok
Update
296513c
[ghstack-poisoned]
@swolchok
Update
845a01e
[ghstack-poisoned]
@swolchok
Update
a92958a
[ghstack-poisoned]
@swolchok
Update
3fa99d6
[ghstack-poisoned]
@swolchok
Update
a6c69a6
[ghstack-poisoned]
@swolchok
Update
3bd6437
[ghstack-poisoned]
@swolchok
Update
675f01b
[ghstack-poisoned]
@swolchok
Update
5f3a768
[ghstack-poisoned]
@swolchok
Update
9fdebee
[ghstack-poisoned]
@swolchok
Update
70a7096
[ghstack-poisoned]
@swolchok
Update
337dc23
[ghstack-poisoned]
@swolchok
Update
f388177
[ghstack-poisoned]
@swolchok
Update
2949daf
[ghstack-poisoned]
@swolchok
Update
7347915
[ghstack-poisoned]
@swolchok
Update
1a8481d
[ghstack-poisoned]
@swolchok
Update
e48e816
[ghstack-poisoned]
@swolchok
Update
3351d50
[ghstack-poisoned]
@swolchok
Update
0102e25
[ghstack-poisoned]
@swolchok
Update
956f8a5
[ghstack-poisoned]
@swolchok
Update
9f7f0c1
[ghstack-poisoned]
@swolchok
Update
a1aeae7
[ghstack-poisoned]
@swolchok
Update
c658163
[ghstack-poisoned]
@swolchok
Update
7e0ccd4
[ghstack-poisoned]
@swolchok
Update
d9cd27c
[ghstack-poisoned]
@swolchok

swolchok commented Mar 18, 2025

Copy link
Copy Markdown
ContributorAuthor

hmm, not good that this is breaking tests.

@swolchok
Update
6c7a39d
[ghstack-poisoned]
@swolchok
Update
784fc13
[ghstack-poisoned]
@swolchok
Update
98f27c1
[ghstack-poisoned]
Base automatically changed from gh/swolchok/369/head to mainMarch 18, 2025 16:24
@swolchok

swolchok commented Mar 18, 2025

Copy link
Copy Markdown
ContributorAuthor

parallelizing copy.out seems to be what's breaking the tests, which is troubling since it's not at all clear why that would happen

@swolchokswolchok mentioned this pull request Mar 18, 2025
Closed
@swolchok
Update
a865349
[ghstack-poisoned]
@swolchok
Update
0b8886f
[ghstack-poisoned]
@swolchokswolchok mentioned this pull request Mar 18, 2025
Merged
@swolchok
swolchok changed the base branch from main to gh/swolchok/378/headMarch 18, 2025 20:49
@swolchok
swolchok marked this pull request as ready for review March 18, 2025 20:53
@swolchokswolchok added the release notes: ops & kernels Changes to the opset and any new / changed kernel implementations label Mar 18, 2025
Base automatically changed from gh/swolchok/378/head to mainMarch 18, 2025 21:39
@swolchok
swolchok merged commit a28e9be into mainMar 18, 2025
@swolchok
swolchok deleted the gh/swolchok/358/head branch March 18, 2025 23:52
oscarandersson8218 pushed a commit to oscarandersson8218/executorch that referenced this pull request Mar 21, 2025
@swolchok@oscarandersson8218
Use parallel_for in elementwise_util (pytorch#9243)
65f7336
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have
exactly two instances of it. If it keeps cropping up, we can add a
utility to make it nicer.)
DannyYuyang-quic pushed a commit to CodeLinaro/executorch that referenced this pull request Apr 2, 2025
@swolchok@DannyYuyang-quic
Use parallel_for in elementwise_util (pytorch#9243)
eeb16c0
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have
exactly two instances of it. If it keeps cropping up, we can add a
utility to make it nicer.)
@swolchok

swolchok commented Apr 13, 2025

Copy link
Copy Markdown
ContributorAuthor

this was for #8932

kedarnath03 pushed a commit to kedarnath03/executorch that referenced this pull request Jun 25, 2025
@swolchok
Use parallel_for in elementwise_util
a07174d
More straightforward rollout.
ghstack-source-id: 33682c4
ghstack-comment-id: 2722704362
Pull Request resolved: pytorch/executorch#9243
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@manuelcandalesmanuelcandalesmanuelcandales approved these changes
@JacobSzwejbkaJacobSzwejbkaAwaiting requested review from JacobSzwejbka
@kimishpatelkimishpatelAwaiting requested review from kimishpatel

Assignees

No one assigned

Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.release notes: ops & kernelsChanges to the opset and any new / changed kernel implementations

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@swolchok@manuelcandales@facebook-github-bot
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all 
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '

Skip to content
Use parallel_for in elementwise_util - #9243
Merged
swolchok merged 162 commits into
mainfrom 
gh/swolchok/358/head
Mar 18, 2025
Merged
Use parallel_for in elementwise_util#9243
swolchok merged 162 commits into
mainfrom 
gh/swolchok/358/head
Conversation
@swolchok
@swolchokswolchok
commented
Mar 13, 2025

Copy link


Copy Markdown

Contributor
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have exactly two instances of it. If it keeps cropping up, we can add a utility to make it nicer.)
swolchok
added 30 commits
March 4, 2025 11:35
@swolchok
Update
d0b11e8
[ghstack-poisoned]
@swolchok
Update
9437be1
[ghstack-poisoned]
@swolchok
Update
643e10e
[ghstack-poisoned]
@swolchok
Update
6f2842b
[ghstack-poisoned]
@swolchok
Update
e47dfeb
[ghstack-poisoned]
@swolchok
Update
231ebc3
[ghstack-poisoned]
@swolchok
Update
296513c
[ghstack-poisoned]
@swolchok
Update
845a01e
[ghstack-poisoned]
@swolchok
Update
a92958a
[ghstack-poisoned]
@swolchok
Update
3fa99d6
[ghstack-poisoned]
@swolchok
Update
a6c69a6
[ghstack-poisoned]
@swolchok
Update
3bd6437
[ghstack-poisoned]
@swolchok
Update
675f01b
[ghstack-poisoned]
@swolchok
Update
5f3a768
[ghstack-poisoned]
@swolchok
Update
9fdebee
[ghstack-poisoned]
@swolchok
Update
70a7096
[ghstack-poisoned]
@swolchok
Update
337dc23
[ghstack-poisoned]
@swolchok
Update
f388177
[ghstack-poisoned]
@swolchok
Update
2949daf
[ghstack-poisoned]
@swolchok
Update
7347915
[ghstack-poisoned]
@swolchok
Update
1a8481d
[ghstack-poisoned]
@swolchok
Update
e48e816
[ghstack-poisoned]
@swolchok
Update
3351d50
[ghstack-poisoned]
@swolchok
Update
0102e25
[ghstack-poisoned]
@swolchok
Update
956f8a5
[ghstack-poisoned]
@swolchok
Update
9f7f0c1
[ghstack-poisoned]
@swolchok
Update
a1aeae7
[ghstack-poisoned]
@swolchok
Update
c658163
[ghstack-poisoned]
@swolchok
Update
7e0ccd4
[ghstack-poisoned]
@swolchok
Update
d9cd27c
[ghstack-poisoned]
@swolchok
swolchok
commented
Mar 18, 2025

Copy link


Copy Markdown

ContributorAuthor
hmm, not good that this is breaking tests.
@swolchok
Update
6c7a39d
[ghstack-poisoned]
@swolchok
Update
784fc13
[ghstack-poisoned]
@swolchok
Update
98f27c1
[ghstack-poisoned]

Base automatically changed from
gh/swolchok/369/head to
mainMarch 18, 2025 16:24
@swolchok
swolchok
commented
Mar 18, 2025

Copy link


Copy Markdown

ContributorAuthor
parallelizing copy.out seems to be what's breaking the tests, which is troubling since it's not at all clear why that would happen
@swolchokswolchok
mentioned this pull request
Mar 18, 2025
 Closed

@swolchok
Update
a865349
[ghstack-poisoned]
@swolchok
Update
0b8886f
[ghstack-poisoned]
@swolchokswolchok
mentioned this pull request
Mar 18, 2025
 Merged

@swolchok
swolchok
changed the base branch from
main
to
gh/swolchok/378/headMarch 18, 2025 20:49
@swolchok
swolchok
marked this pull request as ready for review
March 18, 2025 20:53
@swolchokswolchok
added
the 
release notes: ops & kernels
Changes to the opset and any new / changed kernel implementations
label
Mar 18, 2025

Base automatically changed from
gh/swolchok/378/head to
mainMarch 18, 2025 21:39
@swolchok
swolchok
merged commit a28e9be
into
mainMar 18, 2025
@swolchok
swolchok
deleted the

gh/swolchok/358/head

branch
March 18, 2025 23:52
oscarandersson8218 pushed a commit
to oscarandersson8218/executorch
that referenced
this pull request
Mar 21, 2025
@swolchok@oscarandersson8218
Use parallel_for in elementwise_util (pytorch#9243)
65f7336
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have
exactly two instances of it. If it keeps cropping up, we can add a
utility to make it nicer.)
DannyYuyang-quic pushed a commit
to CodeLinaro/executorch
that referenced
this pull request
Apr 2, 2025
@swolchok@DannyYuyang-quic
Use parallel_for in elementwise_util (pytorch#9243)
eeb16c0
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have
exactly two instances of it. If it keeps cropping up, we can add a
utility to make it nicer.)
@swolchok
swolchok
commented
Apr 13, 2025

Copy link


Copy Markdown

ContributorAuthor
this was for #8932
kedarnath03 pushed a commit
to kedarnath03/executorch
that referenced
this pull request
Jun 25, 2025
@swolchok
Use parallel_for in elementwise_util
a07174d
More straightforward rollout.
ghstack-source-id: 33682c4
ghstack-comment-id: 2722704362
Pull Request resolved: pytorch/executorch#9243
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@manuelcandalesmanuelcandalesmanuelcandales approved these changes
@JacobSzwejbkaJacobSzwejbkaAwaiting requested review from JacobSzwejbka
@kimishpatelkimishpatelAwaiting requested review from kimishpatel

Assignees


No one assigned


Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.release notes: ops & kernelsChanges to the opset and any new / changed kernel implementations

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@swolchok@manuelcandales@facebook-github-bot

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix");
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Use parallel_for in elementwise_util - #9243
Merged
swolchok merged 162 commits into
mainfrom 
gh/swolchok/358/head
Mar 18, 2025
Merged
Use parallel_for in elementwise_util#9243
swolchok merged 162 commits into
mainfrom 
gh/swolchok/358/head
Conversation
@swolchok
@swolchokswolchok
commented
Mar 13, 2025

Copy link


Copy Markdown

Contributor
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have exactly two instances of it. If it keeps cropping up, we can add a utility to make it nicer.)
swolchok
added 30 commits
March 4, 2025 11:35
@swolchok
Update
d0b11e8
[ghstack-poisoned]
@swolchok
Update
9437be1
[ghstack-poisoned]
@swolchok
Update
643e10e
[ghstack-poisoned]
@swolchok
Update
6f2842b
[ghstack-poisoned]
@swolchok
Update
e47dfeb
[ghstack-poisoned]
@swolchok
Update
231ebc3
[ghstack-poisoned]
@swolchok
Update
296513c
[ghstack-poisoned]
@swolchok
Update
845a01e
[ghstack-poisoned]
@swolchok
Update
a92958a
[ghstack-poisoned]
@swolchok
Update
3fa99d6
[ghstack-poisoned]
@swolchok
Update
a6c69a6
[ghstack-poisoned]
@swolchok
Update
3bd6437
[ghstack-poisoned]
@swolchok
Update
675f01b
[ghstack-poisoned]
@swolchok
Update
5f3a768
[ghstack-poisoned]
@swolchok
Update
9fdebee
[ghstack-poisoned]
@swolchok
Update
70a7096
[ghstack-poisoned]
@swolchok
Update
337dc23
[ghstack-poisoned]
@swolchok
Update
f388177
[ghstack-poisoned]
@swolchok
Update
2949daf
[ghstack-poisoned]
@swolchok
Update
7347915
[ghstack-poisoned]
@swolchok
Update
1a8481d
[ghstack-poisoned]
@swolchok
Update
e48e816
[ghstack-poisoned]
@swolchok
Update
3351d50
[ghstack-poisoned]
@swolchok
Update
0102e25
[ghstack-poisoned]
@swolchok
Update
956f8a5
[ghstack-poisoned]
@swolchok
Update
9f7f0c1
[ghstack-poisoned]
@swolchok
Update
a1aeae7
[ghstack-poisoned]
@swolchok
Update
c658163
[ghstack-poisoned]
@swolchok
Update
7e0ccd4
[ghstack-poisoned]
@swolchok
Update
d9cd27c
[ghstack-poisoned]
@swolchok
swolchok
commented
Mar 18, 2025

Copy link


Copy Markdown

ContributorAuthor
hmm, not good that this is breaking tests.
@swolchok
Update
6c7a39d
[ghstack-poisoned]
@swolchok
Update
784fc13
[ghstack-poisoned]
@swolchok
Update
98f27c1
[ghstack-poisoned]

Base automatically changed from
gh/swolchok/369/head to
mainMarch 18, 2025 16:24
@swolchok
swolchok
commented
Mar 18, 2025

Copy link


Copy Markdown

ContributorAuthor
parallelizing copy.out seems to be what's breaking the tests, which is troubling since it's not at all clear why that would happen
@swolchokswolchok
mentioned this pull request
Mar 18, 2025
 Closed

@swolchok
Update
a865349
[ghstack-poisoned]
@swolchok
Update
0b8886f
[ghstack-poisoned]
@swolchokswolchok
mentioned this pull request
Mar 18, 2025
 Merged

@swolchok
swolchok
changed the base branch from
main
to
gh/swolchok/378/headMarch 18, 2025 20:49
@swolchok
swolchok
marked this pull request as ready for review
March 18, 2025 20:53
@swolchokswolchok
added
the 
release notes: ops & kernels
Changes to the opset and any new / changed kernel implementations
label
Mar 18, 2025

Base automatically changed from
gh/swolchok/378/head to
mainMarch 18, 2025 21:39
@swolchok
swolchok
merged commit a28e9be
into
mainMar 18, 2025
@swolchok
swolchok
deleted the

gh/swolchok/358/head

branch
March 18, 2025 23:52
oscarandersson8218 pushed a commit
to oscarandersson8218/executorch
that referenced
this pull request
Mar 21, 2025
@swolchok@oscarandersson8218
Use parallel_for in elementwise_util (pytorch#9243)
65f7336
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have
exactly two instances of it. If it keeps cropping up, we can add a
utility to make it nicer.)
DannyYuyang-quic pushed a commit
to CodeLinaro/executorch
that referenced
this pull request
Apr 2, 2025
@swolchok@DannyYuyang-quic
Use parallel_for in elementwise_util (pytorch#9243)
eeb16c0
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have
exactly two instances of it. If it keeps cropping up, we can add a
utility to make it nicer.)
@swolchok
swolchok
commented
Apr 13, 2025

Copy link


Copy Markdown

ContributorAuthor
this was for #8932
kedarnath03 pushed a commit
to kedarnath03/executorch
that referenced
this pull request
Jun 25, 2025
@swolchok
Use parallel_for in elementwise_util
a07174d
More straightforward rollout.
ghstack-source-id: 33682c4
ghstack-comment-id: 2722704362
Pull Request resolved: pytorch/executorch#9243
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@manuelcandalesmanuelcandalesmanuelcandales approved these changes
@JacobSzwejbkaJacobSzwejbkaAwaiting requested review from JacobSzwejbka
@kimishpatelkimishpatelAwaiting requested review from kimishpatel

Assignees


No one assigned


Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.release notes: ops & kernelsChanges to the opset and any new / changed kernel implementations

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@swolchok@manuelcandales@facebook-github-bot

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms");
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Use parallel_for in elementwise_util - #9243
Merged
swolchok merged 162 commits into
mainfrom 
gh/swolchok/358/head
Mar 18, 2025
Merged
Use parallel_for in elementwise_util#9243
swolchok merged 162 commits into
mainfrom 
gh/swolchok/358/head
Conversation
@swolchok
@swolchokswolchok
commented
Mar 13, 2025

Copy link


Copy Markdown

Contributor
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have exactly two instances of it. If it keeps cropping up, we can add a utility to make it nicer.)
swolchok
added 30 commits
March 4, 2025 11:35
@swolchok
Update
d0b11e8
[ghstack-poisoned]
@swolchok
Update
9437be1
[ghstack-poisoned]
@swolchok
Update
643e10e
[ghstack-poisoned]
@swolchok
Update
6f2842b
[ghstack-poisoned]
@swolchok
Update
e47dfeb
[ghstack-poisoned]
@swolchok
Update
231ebc3
[ghstack-poisoned]
@swolchok
Update
296513c
[ghstack-poisoned]
@swolchok
Update
845a01e
[ghstack-poisoned]
@swolchok
Update
a92958a
[ghstack-poisoned]
@swolchok
Update
3fa99d6
[ghstack-poisoned]
@swolchok
Update
a6c69a6
[ghstack-poisoned]
@swolchok
Update
3bd6437
[ghstack-poisoned]
@swolchok
Update
675f01b
[ghstack-poisoned]
@swolchok
Update
5f3a768
[ghstack-poisoned]
@swolchok
Update
9fdebee
[ghstack-poisoned]
@swolchok
Update
70a7096
[ghstack-poisoned]
@swolchok
Update
337dc23
[ghstack-poisoned]
@swolchok
Update
f388177
[ghstack-poisoned]
@swolchok
Update
2949daf
[ghstack-poisoned]
@swolchok
Update
7347915
[ghstack-poisoned]
@swolchok
Update
1a8481d
[ghstack-poisoned]
@swolchok
Update
e48e816
[ghstack-poisoned]
@swolchok
Update
3351d50
[ghstack-poisoned]
@swolchok
Update
0102e25
[ghstack-poisoned]
@swolchok
Update
956f8a5
[ghstack-poisoned]
@swolchok
Update
9f7f0c1
[ghstack-poisoned]
@swolchok
Update
a1aeae7
[ghstack-poisoned]
@swolchok
Update
c658163
[ghstack-poisoned]
@swolchok
Update
7e0ccd4
[ghstack-poisoned]
@swolchok
Update
d9cd27c
[ghstack-poisoned]
@swolchok
swolchok
commented
Mar 18, 2025

Copy link


Copy Markdown

ContributorAuthor
hmm, not good that this is breaking tests.
@swolchok
Update
6c7a39d
[ghstack-poisoned]
@swolchok
Update
784fc13
[ghstack-poisoned]
@swolchok
Update
98f27c1
[ghstack-poisoned]

Base automatically changed from
gh/swolchok/369/head to
mainMarch 18, 2025 16:24
@swolchok
swolchok
commented
Mar 18, 2025

Copy link


Copy Markdown

ContributorAuthor
parallelizing copy.out seems to be what's breaking the tests, which is troubling since it's not at all clear why that would happen
@swolchokswolchok
mentioned this pull request
Mar 18, 2025
 Closed

@swolchok
Update
a865349
[ghstack-poisoned]
@swolchok
Update
0b8886f
[ghstack-poisoned]
@swolchokswolchok
mentioned this pull request
Mar 18, 2025
 Merged

@swolchok
swolchok
changed the base branch from
main
to
gh/swolchok/378/headMarch 18, 2025 20:49
@swolchok
swolchok
marked this pull request as ready for review
March 18, 2025 20:53
@swolchokswolchok
added
the 
release notes: ops & kernels
Changes to the opset and any new / changed kernel implementations
label
Mar 18, 2025

Base automatically changed from
gh/swolchok/378/head to
mainMarch 18, 2025 21:39
@swolchok
swolchok
merged commit a28e9be
into
mainMar 18, 2025
@swolchok
swolchok
deleted the

gh/swolchok/358/head

branch
March 18, 2025 23:52
oscarandersson8218 pushed a commit
to oscarandersson8218/executorch
that referenced
this pull request
Mar 21, 2025
@swolchok@oscarandersson8218
Use parallel_for in elementwise_util (pytorch#9243)
65f7336
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have
exactly two instances of it. If it keeps cropping up, we can add a
utility to make it nicer.)
DannyYuyang-quic pushed a commit
to CodeLinaro/executorch
that referenced
this pull request
Apr 2, 2025
@swolchok@DannyYuyang-quic
Use parallel_for in elementwise_util (pytorch#9243)
eeb16c0
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have
exactly two instances of it. If it keeps cropping up, we can add a
utility to make it nicer.)
@swolchok
swolchok
commented
Apr 13, 2025

Copy link


Copy Markdown

ContributorAuthor
this was for #8932
kedarnath03 pushed a commit
to kedarnath03/executorch
that referenced
this pull request
Jun 25, 2025
@swolchok
Use parallel_for in elementwise_util
a07174d
More straightforward rollout.
ghstack-source-id: 33682c4
ghstack-comment-id: 2722704362
Pull Request resolved: pytorch/executorch#9243
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@manuelcandalesmanuelcandalesmanuelcandales approved these changes
@JacobSzwejbkaJacobSzwejbkaAwaiting requested review from JacobSzwejbka
@kimishpatelkimishpatelAwaiting requested review from kimishpatel

Assignees


No one assigned


Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.release notes: ops & kernelsChanges to the opset and any new / changed kernel implementations

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@swolchok@manuelcandales@facebook-github-bot

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links");
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '

Skip to content
Use parallel_for in elementwise_util - #9243
Merged
swolchok merged 162 commits into
mainfrom 
gh/swolchok/358/head
Mar 18, 2025
Merged
Use parallel_for in elementwise_util#9243
swolchok merged 162 commits into
mainfrom 
gh/swolchok/358/head
Conversation
@swolchok
@swolchokswolchok
commented
Mar 13, 2025

Copy link


Copy Markdown

Contributor
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have exactly two instances of it. If it keeps cropping up, we can add a utility to make it nicer.)
swolchok
added 30 commits
March 4, 2025 11:35
@swolchok
Update
d0b11e8
[ghstack-poisoned]
@swolchok
Update
9437be1
[ghstack-poisoned]
@swolchok
Update
643e10e
[ghstack-poisoned]
@swolchok
Update
6f2842b
[ghstack-poisoned]
@swolchok
Update
e47dfeb
[ghstack-poisoned]
@swolchok
Update
231ebc3
[ghstack-poisoned]
@swolchok
Update
296513c
[ghstack-poisoned]
@swolchok
Update
845a01e
[ghstack-poisoned]
@swolchok
Update
a92958a
[ghstack-poisoned]
@swolchok
Update
3fa99d6
[ghstack-poisoned]
@swolchok
Update
a6c69a6
[ghstack-poisoned]
@swolchok
Update
3bd6437
[ghstack-poisoned]
@swolchok
Update
675f01b
[ghstack-poisoned]
@swolchok
Update
5f3a768
[ghstack-poisoned]
@swolchok
Update
9fdebee
[ghstack-poisoned]
@swolchok
Update
70a7096
[ghstack-poisoned]
@swolchok
Update
337dc23
[ghstack-poisoned]
@swolchok
Update
f388177
[ghstack-poisoned]
@swolchok
Update
2949daf
[ghstack-poisoned]
@swolchok
Update
7347915
[ghstack-poisoned]
@swolchok
Update
1a8481d
[ghstack-poisoned]
@swolchok
Update
e48e816
[ghstack-poisoned]
@swolchok
Update
3351d50
[ghstack-poisoned]
@swolchok
Update
0102e25
[ghstack-poisoned]
@swolchok
Update
956f8a5
[ghstack-poisoned]
@swolchok
Update
9f7f0c1
[ghstack-poisoned]
@swolchok
Update
a1aeae7
[ghstack-poisoned]
@swolchok
Update
c658163
[ghstack-poisoned]
@swolchok
Update
7e0ccd4
[ghstack-poisoned]
@swolchok
Update
d9cd27c
[ghstack-poisoned]
@swolchok
swolchok
commented
Mar 18, 2025

Copy link


Copy Markdown

ContributorAuthor
hmm, not good that this is breaking tests.
@swolchok
Update
6c7a39d
[ghstack-poisoned]
@swolchok
Update
784fc13
[ghstack-poisoned]
@swolchok
Update
98f27c1
[ghstack-poisoned]

Base automatically changed from
gh/swolchok/369/head to
mainMarch 18, 2025 16:24
@swolchok
swolchok
commented
Mar 18, 2025

Copy link


Copy Markdown

ContributorAuthor
parallelizing copy.out seems to be what's breaking the tests, which is troubling since it's not at all clear why that would happen
@swolchokswolchok
mentioned this pull request
Mar 18, 2025
 Closed

@swolchok
Update
a865349
[ghstack-poisoned]
@swolchok
Update
0b8886f
[ghstack-poisoned]
@swolchokswolchok
mentioned this pull request
Mar 18, 2025
 Merged

@swolchok
swolchok
changed the base branch from
main
to
gh/swolchok/378/headMarch 18, 2025 20:49
@swolchok
swolchok
marked this pull request as ready for review
March 18, 2025 20:53
@swolchokswolchok
added
the 
release notes: ops & kernels
Changes to the opset and any new / changed kernel implementations
label
Mar 18, 2025

Base automatically changed from
gh/swolchok/378/head to
mainMarch 18, 2025 21:39
@swolchok
swolchok
merged commit a28e9be
into
mainMar 18, 2025
@swolchok
swolchok
deleted the

gh/swolchok/358/head

branch
March 18, 2025 23:52
oscarandersson8218 pushed a commit
to oscarandersson8218/executorch
that referenced
this pull request
Mar 21, 2025
@swolchok@oscarandersson8218
Use parallel_for in elementwise_util (pytorch#9243)
65f7336
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have
exactly two instances of it. If it keeps cropping up, we can add a
utility to make it nicer.)
DannyYuyang-quic pushed a commit
to CodeLinaro/executorch
that referenced
this pull request
Apr 2, 2025
@swolchok@DannyYuyang-quic
Use parallel_for in elementwise_util (pytorch#9243)
eeb16c0
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have
exactly two instances of it. If it keeps cropping up, we can add a
utility to make it nicer.)
@swolchok
swolchok
commented
Apr 13, 2025

Copy link


Copy Markdown

ContributorAuthor
this was for #8932
kedarnath03 pushed a commit
to kedarnath03/executorch
that referenced
this pull request
Jun 25, 2025
@swolchok
Use parallel_for in elementwise_util
a07174d
More straightforward rollout.
ghstack-source-id: 33682c4
ghstack-comment-id: 2722704362
Pull Request resolved: pytorch/executorch#9243
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@manuelcandalesmanuelcandalesmanuelcandales approved these changes
@JacobSzwejbkaJacobSzwejbkaAwaiting requested review from JacobSzwejbka
@kimishpatelkimishpatelAwaiting requested review from kimishpatel

Assignees


No one assigned


Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.release notes: ops & kernelsChanges to the opset and any new / changed kernel implementations

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@swolchok@manuelcandales@facebook-github-bot

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default");
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Use parallel_for in elementwise_util - #9243
Merged
swolchok merged 162 commits into
mainfrom 
gh/swolchok/358/head
Mar 18, 2025
Merged
Use parallel_for in elementwise_util#9243
swolchok merged 162 commits into
mainfrom 
gh/swolchok/358/head
Conversation
@swolchok
@swolchokswolchok
commented
Mar 13, 2025

Copy link


Copy Markdown

Contributor
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have exactly two instances of it. If it keeps cropping up, we can add a utility to make it nicer.)
swolchok
added 30 commits
March 4, 2025 11:35
@swolchok
Update
d0b11e8
[ghstack-poisoned]
@swolchok
Update
9437be1
[ghstack-poisoned]
@swolchok
Update
643e10e
[ghstack-poisoned]
@swolchok
Update
6f2842b
[ghstack-poisoned]
@swolchok
Update
e47dfeb
[ghstack-poisoned]
@swolchok
Update
231ebc3
[ghstack-poisoned]
@swolchok
Update
296513c
[ghstack-poisoned]
@swolchok
Update
845a01e
[ghstack-poisoned]
@swolchok
Update
a92958a
[ghstack-poisoned]
@swolchok
Update
3fa99d6
[ghstack-poisoned]
@swolchok
Update
a6c69a6
[ghstack-poisoned]
@swolchok
Update
3bd6437
[ghstack-poisoned]
@swolchok
Update
675f01b
[ghstack-poisoned]
@swolchok
Update
5f3a768
[ghstack-poisoned]
@swolchok
Update
9fdebee
[ghstack-poisoned]
@swolchok
Update
70a7096
[ghstack-poisoned]
@swolchok
Update
337dc23
[ghstack-poisoned]
@swolchok
Update
f388177
[ghstack-poisoned]
@swolchok
Update
2949daf
[ghstack-poisoned]
@swolchok
Update
7347915
[ghstack-poisoned]
@swolchok
Update
1a8481d
[ghstack-poisoned]
@swolchok
Update
e48e816
[ghstack-poisoned]
@swolchok
Update
3351d50
[ghstack-poisoned]
@swolchok
Update
0102e25
[ghstack-poisoned]
@swolchok
Update
956f8a5
[ghstack-poisoned]
@swolchok
Update
9f7f0c1
[ghstack-poisoned]
@swolchok
Update
a1aeae7
[ghstack-poisoned]
@swolchok
Update
c658163
[ghstack-poisoned]
@swolchok
Update
7e0ccd4
[ghstack-poisoned]
@swolchok
Update
d9cd27c
[ghstack-poisoned]
@swolchok
swolchok
commented
Mar 18, 2025

Copy link


Copy Markdown

ContributorAuthor
hmm, not good that this is breaking tests.
@swolchok
Update
6c7a39d
[ghstack-poisoned]
@swolchok
Update
784fc13
[ghstack-poisoned]
@swolchok
Update
98f27c1
[ghstack-poisoned]

Base automatically changed from
gh/swolchok/369/head to
mainMarch 18, 2025 16:24
@swolchok
swolchok
commented
Mar 18, 2025

Copy link


Copy Markdown

ContributorAuthor
parallelizing copy.out seems to be what's breaking the tests, which is troubling since it's not at all clear why that would happen
@swolchokswolchok
mentioned this pull request
Mar 18, 2025
 Closed

@swolchok
Update
a865349
[ghstack-poisoned]
@swolchok
Update
0b8886f
[ghstack-poisoned]
@swolchokswolchok
mentioned this pull request
Mar 18, 2025
 Merged

@swolchok
swolchok
changed the base branch from
main
to
gh/swolchok/378/headMarch 18, 2025 20:49
@swolchok
swolchok
marked this pull request as ready for review
March 18, 2025 20:53
@swolchokswolchok
added
the 
release notes: ops & kernels
Changes to the opset and any new / changed kernel implementations
label
Mar 18, 2025

Base automatically changed from
gh/swolchok/378/head to
mainMarch 18, 2025 21:39
@swolchok
swolchok
merged commit a28e9be
into
mainMar 18, 2025
@swolchok
swolchok
deleted the

gh/swolchok/358/head

branch
March 18, 2025 23:52
oscarandersson8218 pushed a commit
to oscarandersson8218/executorch
that referenced
this pull request
Mar 21, 2025
@swolchok@oscarandersson8218
Use parallel_for in elementwise_util (pytorch#9243)
65f7336
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have
exactly two instances of it. If it keeps cropping up, we can add a
utility to make it nicer.)
DannyYuyang-quic pushed a commit
to CodeLinaro/executorch
that referenced
this pull request
Apr 2, 2025
@swolchok@DannyYuyang-quic
Use parallel_for in elementwise_util (pytorch#9243)
eeb16c0
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have
exactly two instances of it. If it keeps cropping up, we can add a
utility to make it nicer.)
@swolchok
swolchok
commented
Apr 13, 2025

Copy link


Copy Markdown

ContributorAuthor
this was for #8932
kedarnath03 pushed a commit
to kedarnath03/executorch
that referenced
this pull request
Jun 25, 2025
@swolchok
Use parallel_for in elementwise_util
a07174d
More straightforward rollout.
ghstack-source-id: 33682c4
ghstack-comment-id: 2722704362
Pull Request resolved: pytorch/executorch#9243
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@manuelcandalesmanuelcandalesmanuelcandales approved these changes
@JacobSzwejbkaJacobSzwejbkaAwaiting requested review from JacobSzwejbka
@kimishpatelkimishpatelAwaiting requested review from kimishpatel

Assignees


No one assigned


Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.release notes: ops & kernelsChanges to the opset and any new / changed kernel implementations

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@swolchok@manuelcandales@facebook-github-bot

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers");
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Use parallel_for in elementwise_util - #9243
Merged
swolchok merged 162 commits into
mainfrom 
gh/swolchok/358/head
Mar 18, 2025
Merged
Use parallel_for in elementwise_util#9243
swolchok merged 162 commits into
mainfrom 
gh/swolchok/358/head
Conversation
@swolchok
@swolchokswolchok
commented
Mar 13, 2025

Copy link


Copy Markdown

Contributor
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have exactly two instances of it. If it keeps cropping up, we can add a utility to make it nicer.)
swolchok
added 30 commits
March 4, 2025 11:35
@swolchok
Update
d0b11e8
[ghstack-poisoned]
@swolchok
Update
9437be1
[ghstack-poisoned]
@swolchok
Update
643e10e
[ghstack-poisoned]
@swolchok
Update
6f2842b
[ghstack-poisoned]
@swolchok
Update
e47dfeb
[ghstack-poisoned]
@swolchok
Update
231ebc3
[ghstack-poisoned]
@swolchok
Update
296513c
[ghstack-poisoned]
@swolchok
Update
845a01e
[ghstack-poisoned]
@swolchok
Update
a92958a
[ghstack-poisoned]
@swolchok
Update
3fa99d6
[ghstack-poisoned]
@swolchok
Update
a6c69a6
[ghstack-poisoned]
@swolchok
Update
3bd6437
[ghstack-poisoned]
@swolchok
Update
675f01b
[ghstack-poisoned]
@swolchok
Update
5f3a768
[ghstack-poisoned]
@swolchok
Update
9fdebee
[ghstack-poisoned]
@swolchok
Update
70a7096
[ghstack-poisoned]
@swolchok
Update
337dc23
[ghstack-poisoned]
@swolchok
Update
f388177
[ghstack-poisoned]
@swolchok
Update
2949daf
[ghstack-poisoned]
@swolchok
Update
7347915
[ghstack-poisoned]
@swolchok
Update
1a8481d
[ghstack-poisoned]
@swolchok
Update
e48e816
[ghstack-poisoned]
@swolchok
Update
3351d50
[ghstack-poisoned]
@swolchok
Update
0102e25
[ghstack-poisoned]
@swolchok
Update
956f8a5
[ghstack-poisoned]
@swolchok
Update
9f7f0c1
[ghstack-poisoned]
@swolchok
Update
a1aeae7
[ghstack-poisoned]
@swolchok
Update
c658163
[ghstack-poisoned]
@swolchok
Update
7e0ccd4
[ghstack-poisoned]
@swolchok
Update
d9cd27c
[ghstack-poisoned]
@swolchok
swolchok
commented
Mar 18, 2025

Copy link


Copy Markdown

ContributorAuthor
hmm, not good that this is breaking tests.
@swolchok
Update
6c7a39d
[ghstack-poisoned]
@swolchok
Update
784fc13
[ghstack-poisoned]
@swolchok
Update
98f27c1
[ghstack-poisoned]

Base automatically changed from
gh/swolchok/369/head to
mainMarch 18, 2025 16:24
@swolchok
swolchok
commented
Mar 18, 2025

Copy link


Copy Markdown

ContributorAuthor
parallelizing copy.out seems to be what's breaking the tests, which is troubling since it's not at all clear why that would happen
@swolchokswolchok
mentioned this pull request
Mar 18, 2025
 Closed

@swolchok
Update
a865349
[ghstack-poisoned]
@swolchok
Update
0b8886f
[ghstack-poisoned]
@swolchokswolchok
mentioned this pull request
Mar 18, 2025
 Merged

@swolchok
swolchok
changed the base branch from
main
to
gh/swolchok/378/headMarch 18, 2025 20:49
@swolchok
swolchok
marked this pull request as ready for review
March 18, 2025 20:53
@swolchokswolchok
added
the 
release notes: ops & kernels
Changes to the opset and any new / changed kernel implementations
label
Mar 18, 2025

Base automatically changed from
gh/swolchok/378/head to
mainMarch 18, 2025 21:39
@swolchok
swolchok
merged commit a28e9be
into
mainMar 18, 2025
@swolchok
swolchok
deleted the

gh/swolchok/358/head

branch
March 18, 2025 23:52
oscarandersson8218 pushed a commit
to oscarandersson8218/executorch
that referenced
this pull request
Mar 21, 2025
@swolchok@oscarandersson8218
Use parallel_for in elementwise_util (pytorch#9243)
65f7336
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have
exactly two instances of it. If it keeps cropping up, we can add a
utility to make it nicer.)
DannyYuyang-quic pushed a commit
to CodeLinaro/executorch
that referenced
this pull request
Apr 2, 2025
@swolchok@DannyYuyang-quic
Use parallel_for in elementwise_util (pytorch#9243)
eeb16c0
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have
exactly two instances of it. If it keeps cropping up, we can add a
utility to make it nicer.)
@swolchok
swolchok
commented
Apr 13, 2025

Copy link


Copy Markdown

ContributorAuthor
this was for #8932
kedarnath03 pushed a commit
to kedarnath03/executorch
that referenced
this pull request
Jun 25, 2025
@swolchok
Use parallel_for in elementwise_util
a07174d
More straightforward rollout.
ghstack-source-id: 33682c4
ghstack-comment-id: 2722704362
Pull Request resolved: pytorch/executorch#9243
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@manuelcandalesmanuelcandalesmanuelcandales approved these changes
@JacobSzwejbkaJacobSzwejbkaAwaiting requested review from JacobSzwejbka
@kimishpatelkimishpatelAwaiting requested review from kimishpatel

Assignees


No one assigned


Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.release notes: ops & kernelsChanges to the opset and any new / changed kernel implementations

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@swolchok@manuelcandales@facebook-github-bot

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode");
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();

Skip to content
Use parallel_for in elementwise_util - #9243
Merged
swolchok merged 162 commits into
mainfrom 
gh/swolchok/358/head
Mar 18, 2025
Merged
Use parallel_for in elementwise_util#9243
swolchok merged 162 commits into
mainfrom 
gh/swolchok/358/head
Conversation
@swolchok
@swolchokswolchok
commented
Mar 13, 2025

Copy link


Copy Markdown

Contributor
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have exactly two instances of it. If it keeps cropping up, we can add a utility to make it nicer.)
swolchok
added 30 commits
March 4, 2025 11:35
@swolchok
Update
d0b11e8
[ghstack-poisoned]
@swolchok
Update
9437be1
[ghstack-poisoned]
@swolchok
Update
643e10e
[ghstack-poisoned]
@swolchok
Update
6f2842b
[ghstack-poisoned]
@swolchok
Update
e47dfeb
[ghstack-poisoned]
@swolchok
Update
231ebc3
[ghstack-poisoned]
@swolchok
Update
296513c
[ghstack-poisoned]
@swolchok
Update
845a01e
[ghstack-poisoned]
@swolchok
Update
a92958a
[ghstack-poisoned]
@swolchok
Update
3fa99d6
[ghstack-poisoned]
@swolchok
Update
a6c69a6
[ghstack-poisoned]
@swolchok
Update
3bd6437
[ghstack-poisoned]
@swolchok
Update
675f01b
[ghstack-poisoned]
@swolchok
Update
5f3a768
[ghstack-poisoned]
@swolchok
Update
9fdebee
[ghstack-poisoned]
@swolchok
Update
70a7096
[ghstack-poisoned]
@swolchok
Update
337dc23
[ghstack-poisoned]
@swolchok
Update
f388177
[ghstack-poisoned]
@swolchok
Update
2949daf
[ghstack-poisoned]
@swolchok
Update
7347915
[ghstack-poisoned]
@swolchok
Update
1a8481d
[ghstack-poisoned]
@swolchok
Update
e48e816
[ghstack-poisoned]
@swolchok
Update
3351d50
[ghstack-poisoned]
@swolchok
Update
0102e25
[ghstack-poisoned]
@swolchok
Update
956f8a5
[ghstack-poisoned]
@swolchok
Update
9f7f0c1
[ghstack-poisoned]
@swolchok
Update
a1aeae7
[ghstack-poisoned]
@swolchok
Update
c658163
[ghstack-poisoned]
@swolchok
Update
7e0ccd4
[ghstack-poisoned]
@swolchok
Update
d9cd27c
[ghstack-poisoned]
@swolchok
swolchok
commented
Mar 18, 2025

Copy link


Copy Markdown

ContributorAuthor
hmm, not good that this is breaking tests.
@swolchok
Update
6c7a39d
[ghstack-poisoned]
@swolchok
Update
784fc13
[ghstack-poisoned]
@swolchok
Update
98f27c1
[ghstack-poisoned]

Base automatically changed from
gh/swolchok/369/head to
mainMarch 18, 2025 16:24
@swolchok
swolchok
commented
Mar 18, 2025

Copy link


Copy Markdown

ContributorAuthor
parallelizing copy.out seems to be what's breaking the tests, which is troubling since it's not at all clear why that would happen
@swolchokswolchok
mentioned this pull request
Mar 18, 2025
 Closed

@swolchok
Update
a865349
[ghstack-poisoned]
@swolchok
Update
0b8886f
[ghstack-poisoned]
@swolchokswolchok
mentioned this pull request
Mar 18, 2025
 Merged

@swolchok
swolchok
changed the base branch from
main
to
gh/swolchok/378/headMarch 18, 2025 20:49
@swolchok
swolchok
marked this pull request as ready for review
March 18, 2025 20:53
@swolchokswolchok
added
the 
release notes: ops & kernels
Changes to the opset and any new / changed kernel implementations
label
Mar 18, 2025

Base automatically changed from
gh/swolchok/378/head to
mainMarch 18, 2025 21:39
@swolchok
swolchok
merged commit a28e9be
into
mainMar 18, 2025
@swolchok
swolchok
deleted the

gh/swolchok/358/head

branch
March 18, 2025 23:52
oscarandersson8218 pushed a commit
to oscarandersson8218/executorch
that referenced
this pull request
Mar 21, 2025
@swolchok@oscarandersson8218
Use parallel_for in elementwise_util (pytorch#9243)
65f7336
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have
exactly two instances of it. If it keeps cropping up, we can add a
utility to make it nicer.)
DannyYuyang-quic pushed a commit
to CodeLinaro/executorch
that referenced
this pull request
Apr 2, 2025
@swolchok@DannyYuyang-quic
Use parallel_for in elementwise_util (pytorch#9243)
eeb16c0
More straightforward rollout.
(Parallelizing over BroadcastIndexesRange is ugly, but so far we have
exactly two instances of it. If it keeps cropping up, we can add a
utility to make it nicer.)
@swolchok
swolchok
commented
Apr 13, 2025

Copy link


Copy Markdown

ContributorAuthor
this was for #8932
kedarnath03 pushed a commit
to kedarnath03/executorch
that referenced
this pull request
Jun 25, 2025
@swolchok
Use parallel_for in elementwise_util
a07174d
More straightforward rollout.
ghstack-source-id: 33682c4
ghstack-comment-id: 2722704362
Pull Request resolved: pytorch/executorch#9243
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@manuelcandalesmanuelcandalesmanuelcandales approved these changes
@JacobSzwejbkaJacobSzwejbkaAwaiting requested review from JacobSzwejbka
@kimishpatelkimishpatelAwaiting requested review from kimishpatel

Assignees


No one assigned


Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.release notes: ops & kernelsChanges to the opset and any new / changed kernel implementations

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@swolchok@manuelcandales@facebook-github-bot