-
Notifications
You must be signed in to change notification settings - Fork 222
Compilation with src and bin tokenizers, genai and oponvino. #3657
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
rasapala
wants to merge
48
commits into
main
Choose a base branch
from
CVS-173265_genai_build
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
48 commits
Select commit
Hold shift + click to select a range
edf7431
Linux compiles with binary
rasapala 664ee82
Fix redhat
rasapala db69ba4
Shared flag
rasapala 7af85c2
fix sdl
rasapala 3945a97
Fix sdl
rasapala 0a29876
Fix windows build
rasapala 0b2d025
Genai from source
rasapala 8362d22
Add deps
rasapala 0b1aada
Fix dockerfile
rasapala b7f82f4
Fix include
rasapala 7c974fd
Fix spell and if
rasapala 1ba6ec8
Spell2
rasapala 28f77cd
Add tokenizers flag
rasapala 5c7bfdc
Merge branch 'main' into CVS-173265_genai_build
rasapala 8ce18c7
Add tokenizers back
rasapala 00baf0c
Fix sdl
rasapala 510edcf
Fix tokenizers
rasapala 6555d76
Merge branch 'main' into CVS-173265_genai_build
rasapala c9d60d0
Add test env
rasapala a150397
Merge branch 'main' into CVS-173265_genai_build
rasapala 58c1fda
Merge branch 'main' into CVS-173265_genai_build
rasapala b846935
Merge branch 'main' into CVS-173265_genai_build
rasapala 983ab94
Fix redhat
rasapala 8fb5571
Fix dockerfile
rasapala 39cc8ef
Add windows build from source
rasapala d4c1573
Test source
rasapala e9c5b6c
Fix workspace changes
rasapala d122b9a
Fix test and ov_bin=0
rasapala 4a423d3
Style
rasapala 86ed5f2
test ubuntu with bin on
rasapala 83b15cd
Fix test
rasapala 8febedb
Remove tokenizer from linking
rasapala 51de4e6
Merge branch 'main' into CVS-173265_genai_build
rasapala a0d714a
SEt default build from bin windows
rasapala 392049a
Cleanup and test bin=1 redhat
rasapala f05db8b
Fix create package
rasapala 070ae9e
Redhat from sources
rasapala 71d07a4
Merge remote-tracking branch 'origin/CVS-173265_genai_build' into CVS…
rasapala 39fa049
Fix makefile ovveride
rasapala fdcbbaf
Fix lib
rasapala e59b295
Fix libs and python
rasapala 7d6a395
Fix flags
rasapala d7afaac
Code review
rasapala 55983ca
Fix package
rasapala 06ae2d0
Fix docker
rasapala f9cf03f
Fix docker 2
rasapala ccaff32
Fix docker
rasapala 568cee3
Fix create
rasapala File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -490,10 +490,6 @@ rules_pkg_dependencies() | |
load("@ovms//third_party/aws-sdk-cpp:aws-sdk-cpp.bzl", "aws_sdk_cpp") | ||
aws_sdk_cpp() | ||
|
||
### OpenVINO GenAI | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Since we remove this we need synchronize this with Mediapipe repository update |
||
load("@ovms//third_party/llm_engine:llm_engine.bzl", "llm_engine") | ||
llm_engine() | ||
|
||
### Libgit2 | ||
load("@ovms//third_party/libgit2:libgit2_engine.bzl", "libgit2_engine") | ||
libgit2_engine() | ||
|
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Idea worth considering:
We could have "build_ov/genai.sh/bat" that will use variables defined in the container (stored in env.txt?).
Then in case of upgrade in OV we wouldn;t have to rebuild build container from scratch - just relaunch this script