-
Notifications
You must be signed in to change notification settings - Fork 958
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
Embed yolo files #831
Open
katsu560
wants to merge
22
commits into
ggerganov:master
Choose a base branch
from
katsu560:embed_yolo_files
base: master
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
Embed yolo files #831
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
6af7435
ggml : add namedobject to GGUF_TYPE for adding files to model file
661588c
yolo : add reading labels and alphabet labels from model file
ecf8043
yolo : add files to gguf file script
33cf5b3
read data from kv string
73a168b
remove NAMEDOBJECT, use key and STRING value
3234fa1
remove dump code
aaa93bc
read file data from tensor
bcf4ec8
add files to kv and tensor data
8d6feac
Merge branch 'ggerganov:master' into embed_yolo_files
katsu560 d13e8ba
Merge branch 'ggerganov:master' into embed_yolo_files
katsu560 2210bb0
Merge branch 'ggerganov:master' into embed_yolo_files
katsu560 2c3603e
load files from model
50d5683
load files from model
9f70ebf
load files from model
e8720f6
add yolo test, making gguf and reading files from gguf
695fbaf
remove debug code, unused code
e18593c
refactor code, fix copying key value, add --force
3f06cef
delete gguf_find_key_array()
7d59c7a
remove gguf_get_tensor_size
54506bd
minor
ggerganov 0f77e0a
minor changes
b00235b
Merge branch 'embed_yolo_files' of https://github.com/katsu560/ggml i…
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
load files from model
- Loading branch information
There are no files selected for viewing
This file contains 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 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 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 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 |
---|---|---|
|
@@ -2305,6 +2305,7 @@ extern "C" { | |
|
||
GGML_API int gguf_get_n_kv(const struct gguf_context * ctx); | ||
GGML_API int gguf_find_key(const struct gguf_context * ctx, const char * key); | ||
GGML_API int gguf_find_key_array(const struct gguf_context * ctx, const char * key, const char * val); | ||
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. No need to add |
||
GGML_API const char * gguf_get_key (const struct gguf_context * ctx, int key_id); | ||
|
||
GGML_API enum gguf_type gguf_get_kv_type (const struct gguf_context * ctx, int key_id); | ||
|
@@ -2333,6 +2334,7 @@ extern "C" { | |
GGML_API size_t gguf_get_tensor_offset(const struct gguf_context * ctx, int i); | ||
GGML_API char * gguf_get_tensor_name (const struct gguf_context * ctx, int i); | ||
GGML_API enum ggml_type gguf_get_tensor_type (const struct gguf_context * ctx, int i); | ||
GGML_API size_t gguf_get_tensor_size (const struct gguf_context * ctx, int i); | ||
|
||
// removes key if it exists | ||
GGML_API void gguf_remove_key(struct gguf_context * ctx, const char * key); | ||
|
This file contains 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
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.
Somehow I didn't notice this before:
gguf_get_tensor_size()
is not needed too. You can instead use:So remove
gguf_get_tensor_size
all togetherThere 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.
okay, i removed gguf_get_tensor_size from ggml.h, ggml.c, yolov3-tiny.cpp.