-
Notifications
You must be signed in to change notification settings - Fork 76
Embedding admission strategy #236
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
Merged
Merged
Changes from all commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
83be6b8
Draft usage of KVCounter.
jiashuy 89d0aa5
Add FrequencyAdmissionStrategy and AdmissionStrategy class.
z52527 79fca8f
Add storage only admission in training(Need to test).
z52527 447e045
Add storage only admission in training Step 2.
z52527 d842f38
Add cache and storage admission in training(Need to test).
z52527 3d3ce8e
Pass Admission Counter to KeyValueTable lookup.
z52527 ad5c5e3
Add has admission flag for dedeup part in input dist.
z52527 c011899
Add test for embedding admssion and fix bug in lookup.
z52527 9b23d43
Fix cache frequency bug.
z52527 d8d679e
Fix some bugs.
z52527 afce824
Rebase Counter table and fix some comment's issues.
z52527 79daf92
Move admit stratedy class to embedding_admission.py.
z52527 438bb8a
Rebase Counter table and move counter init outsite tableoptions.
z52527 c59ba95
Fix some bugs.
z52527 fd6309c
Dump and load correct counter files in dynamic_table_v2
jiashuy fa36cd6
Unit test of counter table's checkpoint.
jiashuy 87fa71f
Decoupling lookup and admission.
z52527 932fc91
Decoupling training and insert.
z52527 0ad108f
Do admission before initalizer.
z52527 d276e62
Add DynamicEmbInitializerArgs for admit strategy.
z52527 33bb2d5
Add Initializer for non-admit embs.
z52527 a2b3e29
Fix circular dependency about initializer class.
z52527 493f23f
Update document about embedding admission
jiashuy cfa84a7
Add admission options to example.py.
z52527 2eef69c
Add comment for admission threshlod.
z52527 31bc96b
Fix segmented unique rebase bugs.
z52527 1ac9ca7
Fix segmented unique rebase bugs step2.
z52527 8d72d14
Fix duplicated check to counter keys in test_embedding_dump_load.py
jiashuy a1dc575
Fix test and format codes
jiashuy 6ac2721
Move create_initializer to initializer.py to unify the creation logic.
z52527 a6f9876
Add score_strategy and admit strategy into get_grouped_key of Dynamic…
z52527 db2810c
Fix admission test assertion for mutli gpus.
z52527 d421858
Integrated initialize_non_admitted_embeddings.
z52527 03e71ae
Pass admit strategy and evict strategy from table to function.
z52527 cf78f0b
Fix bugs.
z52527 9b7e58f
Fix bugs.
z52527 9a87370
Remove some comments.
z52527 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
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.
Uh oh!
There was an error while loading. Please reload this page.