Documentation changes for openvino backend - #14

Merged
ynimmaga merged 12 commits into
openvino_backendfrom
doc_changes
Feb 12, 2025
Merged

Documentation changes for openvino backend#14
ynimmaga merged 12 commits into
openvino_backendfrom
doc_changes

Conversation

@ynimmaga

@ynimmagaynimmaga commented Feb 11, 2025

Copy link
Copy Markdown
Owner
  • Added a README.md for backends/openvino
  • Added a README.md for examples/openvino
  • Deleted the old ReadMe.md for examples/openvino

cavusmustafa
cavusmustafa approved these changes Feb 12, 2025
View reviewed changes

@cavusmustafacavusmustafa left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added my comments for all the changes. The rest looks good to me.

Comment threadbackends/openvino/README.md Outdated
Comment threadbackends/openvino/README.md Outdated
Comment threadbackends/openvino/README.md Outdated
Comment threadbackends/openvino/README.md
Comment threadbackends/openvino/README.md Outdated
Comment threaddocs/source/build-run-openvino.md Outdated
Comment threaddocs/source/build-run-openvino.md Outdated
Comment threaddocs/source/build-run-openvino.md Outdated
Comment threaddocs/source/build-run-openvino.md
Comment threaddocs/source/build-run-openvino.md
ynimmagaand others added 5 commits February 11, 2025 19:02
@ynimmaga@cavusmustafa
Update backends/openvino/README.md
7139bac
Co-authored-by: Mustafa Cavus <mustafa.cavus@intel.com>
@ynimmaga@cavusmustafa
remove enable_wheel from openvino build instructions
58152a2
Co-authored-by: Mustafa Cavus <mustafa.cavus@intel.com>
@ynimmaga@cavusmustafa
Update backends/openvino/README.md
541e44d
Co-authored-by: Mustafa Cavus <mustafa.cavus@intel.com>
@ynimmaga
Updated executorch environment setup in README.md
9968178
@ynimmaga
Update environment setup in build-run-openvino.md
32e7cc7
@ynimmaga
ynimmaga merged commit 26f05f5 into openvino_backendFeb 12, 2025
suryasidd pushed a commit that referenced this pull request Aug 14, 2025
@metascroy
Do not use BNNS copy when dtypes differ in CoreML (pytorch#13018)
43d90e5
BNNS copy crashes the process when the dtypes differ
(pytorch#11714).
With the example in this PR
(pytorch#11714), we crash the
process on main. Here is the stack trace from LLDB:
```
Process 19234 stopped
* thread #1, queue = 'com.apple.main-thread', stop reason = signal SIGABRT
frame #0: 0x0000000190ac9388 libsystem_kernel.dylib`__pthread_kill + 8
libsystem_kernel.dylib`__pthread_kill:
-> 0x190ac9388 <+8>: b.lo 0x190ac93a8 ; <+40>
0x190ac938c <+12>: pacibsp 0x190ac9390 <+16>: stp x29, x30, [sp, #-0x10]!
0x190ac9394 <+20>: mov x29, sp
(lldb) bt
* thread #1, queue = 'com.apple.main-thread', stop reason = signal SIGABRT
* frame #0: 0x0000000190ac9388 libsystem_kernel.dylib`__pthread_kill + 8
frame #1: 0x0000000190b0288c libsystem_pthread.dylib`pthread_kill + 296
frame #2: 0x0000000190a0bc60 libsystem_c.dylib`abort + 124
frame #3: 0x0000000190910174 libsystem_malloc.dylib`malloc_vreport + 892
frame #4: 0x0000000190913c90 libsystem_malloc.dylib`malloc_report + 64
frame #5: 0x000000019091821c libsystem_malloc.dylib`___BUG_IN_CLIENT_OF_LIBMALLOC_POINTER_BEING_FREED_WAS_NOT_ALLOCATED + 32
frame #6: 0x000000019d2f4084 libBNNS.dylib`___lldb_unnamed_symbol1620 + 564
frame #7: 0x000000019d2f5bac libBNNS.dylib`___lldb_unnamed_symbol1628 + 680
frame #8: 0x000000019d69ce48 libBNNS.dylib`BNNSCopy + 616
frame #9: 0x000000030c74d950 _portable_lib.cpython-310-darwin.so`(anonymous namespace)::copy_using_bnns(executorchcoreml::MultiArray const&, executorchcoreml::MultiArray&) + 188
frame #10: 0x000000030c74cfdc _portable_lib.cpython-310-darwin.so`(anonymous namespace)::copy(executorchcoreml::MultiArray const&, executorchcoreml::MultiArray&, executorchcoreml::MultiArray::CopyOptions) + 72
frame #11: 0x000000030c74ceec _portable_lib.cpython-310-darwin.so`executorchcoreml::MultiArray::copy(executorchcoreml::MultiArray&, executorchcoreml::MultiArray::CopyOptions) const + 148
frame #12: 0x000000030c7488d4 _portable_lib.cpython-310-darwin.so`invocation function for block in (anonymous namespace)::copy(MLMultiArray*, executorchcoreml::MultiArray&) + 376
frame #13: 0x000000030c748ac8 _portable_lib.cpython-310-darwin.so`invocation function for block in (anonymous namespace)::copy(MLMultiArray*, executorchcoreml::MultiArray&) + 52
frame #14: 0x000000019ad33f4c CoreML`CoreML::MultiArrayBuffer::getBytesWithHandler(void (void const*, unsigned long) block_pointer) const + 340
frame #15: 0x000000019ad34138 CoreML`-[MLMultiArray(ScopedBufferAccess) getBytesWithHandler:] + 152
frame #16: 0x000000030c7485ec _portable_lib.cpython-310-darwin.so`(anonymous namespace)::copy(MLMultiArray*, executorchcoreml::MultiArray&) + 296
frame #17: 0x000000030c744f68 _portable_lib.cpython-310-darwin.so`(anonymous namespace)::set_outputs(std::__1::vector<executorchcoreml::MultiArray, std::__1::allocator<executorchcoreml::MultiArray>>&, NSArray<MLMultiArray*>*) + 180
```
With this PR, the process succeeds.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cavusmustafacavusmustafacavusmustafa approved these changes
@suryasiddsuryasiddAwaiting requested review from suryasidd

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ynimmaga@cavusmustafa
, '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

Documentation changes for openvino backend - #14

Merged
ynimmaga merged 12 commits into
openvino_backendfrom
doc_changes
Feb 12, 2025
Merged

Documentation changes for openvino backend#14
ynimmaga merged 12 commits into
openvino_backendfrom
doc_changes

Conversation

@ynimmaga

@ynimmagaynimmaga commented Feb 11, 2025

Copy link
Copy Markdown
Owner
  • Added a README.md for backends/openvino
  • Added a README.md for examples/openvino
  • Deleted the old ReadMe.md for examples/openvino

cavusmustafa
cavusmustafa approved these changes Feb 12, 2025
View reviewed changes

@cavusmustafacavusmustafa left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added my comments for all the changes. The rest looks good to me.

Comment threadbackends/openvino/README.md Outdated
Comment threadbackends/openvino/README.md Outdated
Comment threadbackends/openvino/README.md Outdated
Comment threadbackends/openvino/README.md
Comment threadbackends/openvino/README.md Outdated
Comment threaddocs/source/build-run-openvino.md Outdated
Comment threaddocs/source/build-run-openvino.md Outdated
Comment threaddocs/source/build-run-openvino.md Outdated
Comment threaddocs/source/build-run-openvino.md
Comment threaddocs/source/build-run-openvino.md
ynimmagaand others added 5 commits February 11, 2025 19:02
@ynimmaga@cavusmustafa
Update backends/openvino/README.md
7139bac
Co-authored-by: Mustafa Cavus <mustafa.cavus@intel.com>
@ynimmaga@cavusmustafa
remove enable_wheel from openvino build instructions
58152a2
Co-authored-by: Mustafa Cavus <mustafa.cavus@intel.com>
@ynimmaga@cavusmustafa
Update backends/openvino/README.md
541e44d
Co-authored-by: Mustafa Cavus <mustafa.cavus@intel.com>
@ynimmaga
Updated executorch environment setup in README.md
9968178
@ynimmaga
Update environment setup in build-run-openvino.md
32e7cc7
@ynimmaga
ynimmaga merged commit 26f05f5 into openvino_backendFeb 12, 2025
suryasidd pushed a commit that referenced this pull request Aug 14, 2025
@metascroy
Do not use BNNS copy when dtypes differ in CoreML (pytorch#13018)
43d90e5
BNNS copy crashes the process when the dtypes differ
(pytorch#11714).
With the example in this PR
(pytorch#11714), we crash the
process on main. Here is the stack trace from LLDB:
```
Process 19234 stopped
* thread #1, queue = 'com.apple.main-thread', stop reason = signal SIGABRT
frame #0: 0x0000000190ac9388 libsystem_kernel.dylib`__pthread_kill + 8
libsystem_kernel.dylib`__pthread_kill:
-> 0x190ac9388 <+8>: b.lo 0x190ac93a8 ; <+40>
0x190ac938c <+12>: pacibsp 0x190ac9390 <+16>: stp x29, x30, [sp, #-0x10]!
0x190ac9394 <+20>: mov x29, sp
(lldb) bt
* thread #1, queue = 'com.apple.main-thread', stop reason = signal SIGABRT
* frame #0: 0x0000000190ac9388 libsystem_kernel.dylib`__pthread_kill + 8
frame #1: 0x0000000190b0288c libsystem_pthread.dylib`pthread_kill + 296
frame #2: 0x0000000190a0bc60 libsystem_c.dylib`abort + 124
frame #3: 0x0000000190910174 libsystem_malloc.dylib`malloc_vreport + 892
frame #4: 0x0000000190913c90 libsystem_malloc.dylib`malloc_report + 64
frame #5: 0x000000019091821c libsystem_malloc.dylib`___BUG_IN_CLIENT_OF_LIBMALLOC_POINTER_BEING_FREED_WAS_NOT_ALLOCATED + 32
frame #6: 0x000000019d2f4084 libBNNS.dylib`___lldb_unnamed_symbol1620 + 564
frame #7: 0x000000019d2f5bac libBNNS.dylib`___lldb_unnamed_symbol1628 + 680
frame #8: 0x000000019d69ce48 libBNNS.dylib`BNNSCopy + 616
frame #9: 0x000000030c74d950 _portable_lib.cpython-310-darwin.so`(anonymous namespace)::copy_using_bnns(executorchcoreml::MultiArray const&, executorchcoreml::MultiArray&) + 188
frame #10: 0x000000030c74cfdc _portable_lib.cpython-310-darwin.so`(anonymous namespace)::copy(executorchcoreml::MultiArray const&, executorchcoreml::MultiArray&, executorchcoreml::MultiArray::CopyOptions) + 72
frame #11: 0x000000030c74ceec _portable_lib.cpython-310-darwin.so`executorchcoreml::MultiArray::copy(executorchcoreml::MultiArray&, executorchcoreml::MultiArray::CopyOptions) const + 148
frame #12: 0x000000030c7488d4 _portable_lib.cpython-310-darwin.so`invocation function for block in (anonymous namespace)::copy(MLMultiArray*, executorchcoreml::MultiArray&) + 376
frame #13: 0x000000030c748ac8 _portable_lib.cpython-310-darwin.so`invocation function for block in (anonymous namespace)::copy(MLMultiArray*, executorchcoreml::MultiArray&) + 52
frame #14: 0x000000019ad33f4c CoreML`CoreML::MultiArrayBuffer::getBytesWithHandler(void (void const*, unsigned long) block_pointer) const + 340
frame #15: 0x000000019ad34138 CoreML`-[MLMultiArray(ScopedBufferAccess) getBytesWithHandler:] + 152
frame #16: 0x000000030c7485ec _portable_lib.cpython-310-darwin.so`(anonymous namespace)::copy(MLMultiArray*, executorchcoreml::MultiArray&) + 296
frame #17: 0x000000030c744f68 _portable_lib.cpython-310-darwin.so`(anonymous namespace)::set_outputs(std::__1::vector<executorchcoreml::MultiArray, std::__1::allocator<executorchcoreml::MultiArray>>&, NSArray<MLMultiArray*>*) + 180
```
With this PR, the process succeeds.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cavusmustafacavusmustafacavusmustafa approved these changes
@suryasiddsuryasiddAwaiting requested review from suryasidd

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ynimmaga@cavusmustafa
, '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

Documentation changes for openvino backend - #14

Merged
ynimmaga merged 12 commits into
openvino_backendfrom
doc_changes
Feb 12, 2025
Merged

Documentation changes for openvino backend#14
ynimmaga merged 12 commits into
openvino_backendfrom
doc_changes

Conversation

@ynimmaga

@ynimmagaynimmaga commented Feb 11, 2025

Copy link
Copy Markdown
Owner
  • Added a README.md for backends/openvino
  • Added a README.md for examples/openvino
  • Deleted the old ReadMe.md for examples/openvino

cavusmustafa
cavusmustafa approved these changes Feb 12, 2025
View reviewed changes

@cavusmustafacavusmustafa left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added my comments for all the changes. The rest looks good to me.

Comment threadbackends/openvino/README.md Outdated
Comment threadbackends/openvino/README.md Outdated
Comment threadbackends/openvino/README.md Outdated
Comment threadbackends/openvino/README.md
Comment threadbackends/openvino/README.md Outdated
Comment threaddocs/source/build-run-openvino.md Outdated
Comment threaddocs/source/build-run-openvino.md Outdated
Comment threaddocs/source/build-run-openvino.md Outdated
Comment threaddocs/source/build-run-openvino.md
Comment threaddocs/source/build-run-openvino.md
ynimmagaand others added 5 commits February 11, 2025 19:02
@ynimmaga@cavusmustafa
Update backends/openvino/README.md
7139bac
Co-authored-by: Mustafa Cavus <mustafa.cavus@intel.com>
@ynimmaga@cavusmustafa
remove enable_wheel from openvino build instructions
58152a2
Co-authored-by: Mustafa Cavus <mustafa.cavus@intel.com>
@ynimmaga@cavusmustafa
Update backends/openvino/README.md
541e44d
Co-authored-by: Mustafa Cavus <mustafa.cavus@intel.com>
@ynimmaga
Updated executorch environment setup in README.md
9968178
@ynimmaga
Update environment setup in build-run-openvino.md
32e7cc7
@ynimmaga
ynimmaga merged commit 26f05f5 into openvino_backendFeb 12, 2025
suryasidd pushed a commit that referenced this pull request Aug 14, 2025
@metascroy
Do not use BNNS copy when dtypes differ in CoreML (pytorch#13018)
43d90e5
BNNS copy crashes the process when the dtypes differ
(pytorch#11714).
With the example in this PR
(pytorch#11714), we crash the
process on main. Here is the stack trace from LLDB:
```
Process 19234 stopped
* thread #1, queue = 'com.apple.main-thread', stop reason = signal SIGABRT
frame #0: 0x0000000190ac9388 libsystem_kernel.dylib`__pthread_kill + 8
libsystem_kernel.dylib`__pthread_kill:
-> 0x190ac9388 <+8>: b.lo 0x190ac93a8 ; <+40>
0x190ac938c <+12>: pacibsp 0x190ac9390 <+16>: stp x29, x30, [sp, #-0x10]!
0x190ac9394 <+20>: mov x29, sp
(lldb) bt
* thread #1, queue = 'com.apple.main-thread', stop reason = signal SIGABRT
* frame #0: 0x0000000190ac9388 libsystem_kernel.dylib`__pthread_kill + 8
frame #1: 0x0000000190b0288c libsystem_pthread.dylib`pthread_kill + 296
frame #2: 0x0000000190a0bc60 libsystem_c.dylib`abort + 124
frame #3: 0x0000000190910174 libsystem_malloc.dylib`malloc_vreport + 892
frame #4: 0x0000000190913c90 libsystem_malloc.dylib`malloc_report + 64
frame #5: 0x000000019091821c libsystem_malloc.dylib`___BUG_IN_CLIENT_OF_LIBMALLOC_POINTER_BEING_FREED_WAS_NOT_ALLOCATED + 32
frame #6: 0x000000019d2f4084 libBNNS.dylib`___lldb_unnamed_symbol1620 + 564
frame #7: 0x000000019d2f5bac libBNNS.dylib`___lldb_unnamed_symbol1628 + 680
frame #8: 0x000000019d69ce48 libBNNS.dylib`BNNSCopy + 616
frame #9: 0x000000030c74d950 _portable_lib.cpython-310-darwin.so`(anonymous namespace)::copy_using_bnns(executorchcoreml::MultiArray const&, executorchcoreml::MultiArray&) + 188
frame #10: 0x000000030c74cfdc _portable_lib.cpython-310-darwin.so`(anonymous namespace)::copy(executorchcoreml::MultiArray const&, executorchcoreml::MultiArray&, executorchcoreml::MultiArray::CopyOptions) + 72
frame #11: 0x000000030c74ceec _portable_lib.cpython-310-darwin.so`executorchcoreml::MultiArray::copy(executorchcoreml::MultiArray&, executorchcoreml::MultiArray::CopyOptions) const + 148
frame #12: 0x000000030c7488d4 _portable_lib.cpython-310-darwin.so`invocation function for block in (anonymous namespace)::copy(MLMultiArray*, executorchcoreml::MultiArray&) + 376
frame #13: 0x000000030c748ac8 _portable_lib.cpython-310-darwin.so`invocation function for block in (anonymous namespace)::copy(MLMultiArray*, executorchcoreml::MultiArray&) + 52
frame #14: 0x000000019ad33f4c CoreML`CoreML::MultiArrayBuffer::getBytesWithHandler(void (void const*, unsigned long) block_pointer) const + 340
frame #15: 0x000000019ad34138 CoreML`-[MLMultiArray(ScopedBufferAccess) getBytesWithHandler:] + 152
frame #16: 0x000000030c7485ec _portable_lib.cpython-310-darwin.so`(anonymous namespace)::copy(MLMultiArray*, executorchcoreml::MultiArray&) + 296
frame #17: 0x000000030c744f68 _portable_lib.cpython-310-darwin.so`(anonymous namespace)::set_outputs(std::__1::vector<executorchcoreml::MultiArray, std::__1::allocator<executorchcoreml::MultiArray>>&, NSArray<MLMultiArray*>*) + 180
```
With this PR, the process succeeds.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cavusmustafacavusmustafacavusmustafa approved these changes
@suryasiddsuryasiddAwaiting requested review from suryasidd

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ynimmaga@cavusmustafa