Ensure unique node names and enable CPU fallback for OpenVINO#192
Open
zhaixuejun1993 wants to merge 88 commits into
Open
Ensure unique node names and enable CPU fallback for OpenVINO#192zhaixuejun1993 wants to merge 88 commits into
zhaixuejun1993 wants to merge 88 commits into
Conversation
…GGML graph in api compute_model_outputs()
…ntion_pattern_case to easy extand
* added translate_1to1_match_1_input function and updated gelu and tanh translations * Remove unused translation function calls --------- Co-authored-by: Mustafa Cavus <[email protected]>
* OpenVINO backend: refactor VIEW related operation * Enable VIEW handling in following ops * OpenVINO backend does not support GGML_OP_NORM & GGML_OP_L2_NORM with VIEW input accuracy issue from OpenVINO
Enable T5 model for architecture testing in OpenVINO backend
Enable jamba and kimi-linear for architecture tests
…-oss Fix accuracy issue and enable Arctic and Grok for arch tests
* Initiall gemma4 npu support * temp. fix for gemma4 accuracy bug on npu * Remove hardcoded names for npu-fold handling * revert static n tokens for cont translation as it is not needed * removed unused variable
…der cache. Add environment variable GGML_OPENVINO_ENABLE_CACHE (default: YES). When set to NO, the decoder_cache is bypassed and models are rebuilt from the cgraph on every inference call in both dynamic and static compute paths. This is useful for debugging and verifying correctness without caching interference.
…model_env Add GGML_OPENVINO_ENABLE_CACHE env var for decoder cache control
This reverts commit 0d29a9c.
…_log Disable debug log printing in OpenVINO backend
Enable OpenVINO cache support
…g_src to recorde the src ggml tensor for OpenVINO dynamic shape infer
ravi9
force-pushed
the
dev_backend_openvino
branch
from
d4aa38a to
5cdd4f0
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces several improvements and fixes related to tensor source tracking, OpenVINO backend compatibility, and dynamic dimension handling in the
ggmllibrary. The main changes include adding anorg_srcpointer to tensors for tracking original sources, ensuring unique tensor names for OpenVINO graphs, and refining logic for dynamic dimension inference and test configuration.Tensor source tracking improvements:
org_srcpointer to thestruct ggml_tensor, initialized toNULL, to keep track of the original source tensor, especially for in-place operations. This is reflected in both the struct definition and tensor initialization. [1] [2]org_srcfield is set to reference the source tensor, enabling better tracking of tensor origins.OpenVINO backend compatibility:
Dynamic dimension and input detection fixes:
org_srcfield, improving accuracy in identifying input tokens and positions. [1] [2] [3]## OverviewAdditional information
Requirements